Paymahn Moghadasian f7ee20e122 fix: retrieve image attachments and always enable attachments get (#4) ...

* fix: retrieve image attachment bytes via `attachments get -o`

Image files come back from the upstream FilesGet handler as an out-of-band
MCP ImageContent block; the CLI's toolcall adapter extracted only
TextContent, so `attachments get <image_id>` emitted only metadata and
silently dropped the image bytes (even with --raw). Non-image binaries
were unaffected (single base64-in-JSON text envelope).

CLI-layer fix (pkg/handler stays byte-for-byte upstream for clean merges):
- toolcall.InvokeResult captures ImageContent into Result.Images; Invoke
  is now a thin text-only wrapper, so existing callers are unchanged.
- `attachments get` gains -o/--output <path>: decodes the bytes straight
  to a file and prints a small metadata confirmation, keeping stdout free
  of a multi-MB base64 blob. Without -o, image bytes are folded into the
  same {..,"encoding":"base64","content":..} envelope as non-image
  binaries, so `jq -r .content | base64 --decode` works uniformly.
- Update SKILL.md and README with the -o flag and decode recipes.

Adds unit tests for InvokeResult image capture and the attachment
rendering paths (recover-in-envelope, -o writes raw bytes, stdout has no
blob).

* fix: always enable `attachments get` (drop SLACK_MCP_ATTACHMENT_TOOL gate)

Downloading an attachment is a read-only operation, not a mutation like
posting or reacting, so requiring an opt-in env var was needless friction.
The upstream FilesGet handler reads SLACK_MCP_ATTACHMENT_TOOL from the
environment at parse time; the CLI now sets it to "true" in-process before
invoking (same in-process-env pattern as auth/govslack), so `attachments
get` always works. pkg/handler stays byte-for-byte upstream.

Verified live against the real workspace: `attachments get F0000000000`
with no env var now returns Slack `file_not_found` instead of the
"attachment_get_data tool is disabled" gate error.

Docs: move `attachments get` from the write/sensitive section to the read
commands and drop the env-var prefix from recipes (SKILL.md + README).

2026-07-07 12:12:36 -05:00

..

toolcall_test.go

fix: retrieve image attachments and always enable attachments get (#4)

2026-07-07 12:12:36 -05:00

toolcall.go

fix: retrieve image attachments and always enable attachments get (#4)

2026-07-07 12:12:36 -05:00