Skip to content

Fix Slack upload default target comments#668

Open
decofe wants to merge 1 commit into
mainfrom
centaur/fix-slack-upload-default-target-1781811208
Open

Fix Slack upload default target comments#668
decofe wants to merge 1 commit into
mainfrom
centaur/fix-slack-upload-default-target-1781811208

Conversation

@decofe

@decofe decofe commented Jun 18, 2026

Copy link
Copy Markdown
Member

Summary

  • avoid indexing the optional files argument when Slack upload defaults to the current thread
  • keep upload comments attached only to the first normalized upload path
  • propagate blocks-mode thread_key into CENTAUR_THREAD_KEY inside harness-server so shell commands like slack upload <file> can infer the Slack destination from warm or cold sandboxes
  • add regression coverage for the Slack CLI single-file path and blocks-mode thread-key parsing

Prompted by: @Rjected

Tests

  • cargo test --manifest-path crates/harness-server/Cargo.toml
  • PYTHONPATH=$PWD/tools/productivity:$PWD uv run --with pytest --with typer --with slack-sdk --with python-dotenv --with rich --with structlog pytest tools/productivity/slack/tests/test_cli.py

@decofe decofe force-pushed the centaur/fix-slack-upload-default-target-1781811208 branch from be20631 to e347f3f Compare June 18, 2026 19:42
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.

2 participants