Skip to content

Say that saved attachments are marked, and where that is best effort - #561

Merged
robzolkos merged 2 commits into
mainfrom
attachment-docs
Oct 11, 2026
Merged

robzolkos merged 2 commits into
mainfrom
attachment-docs

Conversation

@robzolkos

@robzolkos robzolkos commented Oct 11, 2026 •

Copy link
Copy Markdown
Collaborator

Follow-up to #560, from its review threads:

  • hey attachment save help and docs/cli.md now say every saved file is marked as downloaded from the internet, that a macOS save that can't be marked fails, and that the Windows mark is written only where the drive can hold it.
  • docs/tui.md stops promising the Windows mark everywhere, and says a zip-based document can still open on Linux, in an archive manager at worst.

Docs and help text only; no behavior change.


Summary by cubic

Clarifies that saved attachments are marked as downloaded from the internet on macOS and Windows, and where that marking is best effort, following up on review threads from #560.

  • hey attachment save help and docs/cli.md now say a macOS save that can't be marked fails, and the Windows mark is written only where the drive can hold it; other systems keep no mark.
  • docs/tui.md stops promising the Windows mark everywhere and notes that a zip-based document can still open on Linux, in an archive manager at worst.

Docs and help text only; no behavior change.

Written for commit 049ef90. Summary will update on new commits.

View guided diff Turn on auto-fix

hey attachment save's help and docs/cli.md now say every saved file is
marked as downloaded from the internet, that a macOS save that can't be
marked fails, and that the Windows mark is written only where the drive can
hold it. docs/tui.md stops promising the Windows mark everywhere and says a
zip-based document can still open on Linux, in an archive manager at worst.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@robzolkos
robzolkos requested a review from a team as a code owner October 11, 2026 20:03
Copilot AI balanced review requested due to automatic review settings October 11, 2026 20:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The documentation incorrectly claims all saved attachments are marked, although marking is a no-op outside macOS and Windows.

3 open findings
What changed in this PR

Clarifies attachment download-origin marking and Linux handling of ZIP-based documents.

Changes:

  • Expands attachment save help and agent notes.
  • Updates CLI and TUI documentation with platform-specific behavior.

[!TIP]
If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.

File Description
internal/​cmd/​attachments_save.go Expands save-command help.
docs/​cli.md Documents download-origin marking.
docs/​tui.md Clarifies marking and ZIP-document handling.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/cli.md Outdated
Comment thread docs/tui.md Outdated
Comment thread internal/cmd/attachments_save.go Outdated

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 3 files

Reply with feedback, questions, or to request a fix.

View guided diff | Turn on auto-fix | Re-trigger cubic

Comment thread docs/cli.md Outdated
Comment thread docs/tui.md Outdated
Comment thread internal/cmd/attachments_save.go Outdated
Linux and the other systems keep no record of where a file came from, so
nothing marks a file saved there; the help text and docs said every saved
file was marked.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approved

The documentation accurately reflects the existing platform-specific behavior.

0 open findings

3 resolved since last review

🧠 Review effort: Balanced

@robzolkos
robzolkos merged commit 206753c into main Oct 11, 2026
26 checks passed
@robzolkos
robzolkos deleted the attachment-docs branch October 11, 2026 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants