Skip to content

Add ignore and stop-ignoring actions - #176

Merged
robzolkos merged 2 commits into
mainfrom
feature/email-ignore
Aug 19, 2026
Merged

Add ignore and stop-ignoring actions#176
robzolkos merged 2 commits into
mainfrom
feature/email-ignore

Conversation

@robzolkos

@robzolkosrobzolkos commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • use HEY’s web UI terminology with bulk hey ignore <id>... and hey stop-ignoring <id>... commands
  • back both commands with the SDK’s typed posting mute/unmute operations while keeping internal API terminology out of user-facing help
  • align the TUI with HEY’s - Ignore and + Stop Ignoring shortcuts
  • keep ignored threads visible, label them [Ignored], and offer the context-appropriate reversal action
  • skip redundant TUI requests for threads already in the requested state
  • replace action-label string coupling with explicit posting action effects for remove, seen, ignore, and stop-ignoring transitions
  • update root help, command metadata, README, embedded skill, API coverage, smoke coverage, and the public surface baseline

Validation

  • make check
  • make race-test
  • smoke test package compilation
  • local build and rendered CLI help/command metadata
  • authenticated CLI ignore/stop-ignoring validation on a hey-cli PR notification; verified muted changed to true and then returned to false
  • authenticated TUI validation with - and +; verified the row stayed visible, [Ignored] and contextual help rendered, and the account state was restored

The HEY web UI labels these actions “Ignore” and “Stop Ignoring”; mute/unmute remains only the underlying SDK/API terminology.


Summary by cubic

Add ignore and stop-ignoring actions in CLI and TUI to match HEY’s web UI. Previously “-” muted a thread and removed it; now “-” ignores the thread but keeps it visible with “[Ignored]”, and “+” stops ignoring.

  • CLI: adds hey ignore <id>... and hey stop-ignoring <id>... backed by SDK mute/unmute; requires at least one box item ID; returns concise JSON summaries; curated help, README, .surface, and skills/hey updated.
  • TUI: introduces explicit posting action effects (remove, seen, ignore, stop-ignoring); keeps ignored threads visible and prefixes subjects with “[Ignored]”; shows “- ignore” or “+ stop ignoring” contextually; skips redundant requests when the thread is already in the requested state.
  • API/tests: documents /postings/mutings.json POST/DELETE coverage; adds command unit tests and TUI tests for effects; adds a smoke test that asserts ignored threads remain visible and verifies summaries and state transitions.

Written for commit bcd7073. Summary will update on new commits.

Review in cubic

CopilotAI balanced review requested due to automatic review settings August 19, 2026 05:26

CopilotAI 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.

Pull request overview

Adds CLI and TUI actions for ignoring threads through the typed SDK mute operations.

Changes:

  • Adds bulk ignore and stop-ignoring commands.
  • Adds contextual TUI shortcuts, state labels, and explicit action effects.
  • Updates documentation, metadata, and tests.

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.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.

Show a summary per file
FileDescription
.surfaceAdds commands to the public surface.
API-COVERAGE.mdDocuments mute API coverage.
README.mdDocuments CLI and TUI actions.
skills/hey/SKILL.mdAdds ignore workflows and triggers.
internal/cmd/help.goLists the new commands.
internal/cmd/help_test.goUpdates help expectations.
internal/cmd/ignore.goImplements bulk ignoring.
internal/cmd/ignore_test.goTests both CLI commands.
internal/cmd/root.goRegisters the commands.
internal/cmd/stop_ignoring.goImplements bulk stop-ignoring.
internal/tui/content.goLabels ignored threads.
internal/tui/mail.goAdds TUI actions and effects.
internal/tui/mail_test.goTests TUI behavior.
tests/smoke/ignore_test.goAdds end-to-end state validation.

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

Comment threadtests/smoke/ignore_test.go
CopilotAI review requested due to automatic review settings August 19, 2026 05:30

CopilotAI 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.

Pull request overview

Copilot reviewed 14 out of 14 changed files in this pull request and generated no new comments.

@robzolkos
robzolkos merged commit 4162b5d into mainAug 19, 2026
25 checks passed
@robzolkos
robzolkos deleted the feature/email-ignore branch August 19, 2026 05:36
Sign up for freeto 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

@robzolkos