Skip to content

fix(inbox): show list output as plain text and print questions on Stop - #233

Merged
st0nefish-ci[bot] merged 1 commit into
masterfrom
fix/inbox-visible-output
Nov 5, 2026
Merged

st0nefish-ci[bot] merged 1 commit into
masterfrom
fix/inbox-visible-output

Conversation

@St0nefish

Copy link
Copy Markdown
Owner

Summary

  • /inbox:show, /inbox:questions and /inbox:decisions now inject the tracker output with ! command injection and reply with it as plain text, so it is no longer collapsed behind a tool call (allowed-tools: Bash retained for the injection).
  • The Stop hook now emits the full question list in systemMessage when nothing is running and questions are open (header only if the list is empty). Decisions stay quiet; background work still shows the "still working" line.
  • Stop hook test and README updated; inbox bumped to 1.0.1.
  • A code review ran this session; its one real finding (missing allowed-tools) and the empty-list guard are fixed.

Test plan

  • bash tests/inbox/test-inbox-hook.sh (14/14)
  • bash .github/scripts/validate-all.sh
  • Confirm in a live session that the skills print plain text and the multi-line Stop message renders

🤖 Generated with Claude Code

The show/questions/decisions skills ran the tracker through Bash, so the
output was collapsed behind a tool call. The Stop hook only printed a
one-line banner when the session was stuck on open questions.

- skills inject the tracker output via command injection and reply with it
  as plain text (allowed-tools: Bash retained for the injection)
- Stop hook emits the full question list in systemMessage when nothing is
  running; header only if the list comes back empty
- update Stop hook test and README
- bump inbox to 1.0.1

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@st0nefish-ci
st0nefish-ci Bot enabled auto-merge November 5, 2026 22:34
@st0nefish-ci
st0nefish-ci Bot merged commit 2d052ad into master Nov 5, 2026
8 checks passed
@st0nefish-ci
st0nefish-ci Bot deleted the fix/inbox-visible-output branch November 5, 2026 22:37
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