Skip to content

Scope Slack ETL token rules - #68

Closed
fineas-bot[bot] wants to merge 4 commits into
mainfrom
codex/slack-etl-token-rules
Closed

Scope Slack ETL token rules#68
fineas-bot[bot] wants to merge 4 commits into
mainfrom
codex/slack-etl-token-rules

Conversation

@fineas-bot

@fineas-bot fineas-bot Bot commented Jun 29, 2026

Copy link
Copy Markdown

Summary

  • scope SLACK_ETL_TOKEN to the Slack Web API calls used by Slack ETL
  • keep GET files.slack.com available for ETL attachment downloads
  • add a real Slack tool manifest regression test so the ETL token rules cannot drift back to the search-only shape

Context

Live Slack sync was failing in workflow sandboxes because the iron-control rule for infra-slack-etl-token only swapped the placeholder on /api/search.messages. Slack ETL starts with conversations.list, so Slack received the literal placeholder and returned invalid_auth.

Test

  • docker run --rm -v /Users/fin-sa/centaur-slack-etl-token-rules:/work -w /work/services/api-rs rust:1-bookworm bash -c 'rustup component add rustfmt && cargo fmt --all --check && cargo test -p centaur-perms real_slack_tool_parses_and_translates'

@fineas-bot fineas-bot Bot closed this Jun 29, 2026
@fineas-bot
fineas-bot Bot force-pushed the codex/slack-etl-token-rules branch from f76abc1 to a0abcf5 Compare June 29, 2026 14:23
@fineas-bot fineas-bot Bot reopened this Jun 29, 2026
fineas-bot Bot added a commit that referenced this pull request Jul 12, 2026
Semantic port of approved PR #68 to the current HTTP-secret schema, with exact manifest translation and documentation regressions.
@fineas-bot

fineas-bot Bot commented Jul 12, 2026

Copy link
Copy Markdown
Author

Carried into #76 at reviewed head 46d5932719cfcbb8a8067e68f98a4c7c2c20c853, including the current-schema translated-rule regression and source/generated Slack ETL docs. Closing this older branch prevents a post-review base change; use #76 for review and merge.

@fineas-bot fineas-bot Bot closed this Jul 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant