Skip to content

Backlog: guard reads only a turn's opening message for authorization (#290) - #291

Closed
CryptoJones wants to merge 1 commit into
mainfrom
fix/guard-midturn-authorization
Closed

Backlog: guard reads only a turn's opening message for authorization (#290)#291
CryptoJones wants to merge 1 commit into
mainfrom
fix/guard-midturn-authorization

Conversation

@CryptoJones

Copy link
Copy Markdown
Owner

Adds the BACKLOG.md entry for #290, keeping the backlog and the Issues tab in sync.

No code change — this is the backlog half of the pair.

The bug, briefly: omind guard classifies authorization from the OPENING message of a turn. Claude Code delivers messages the user sends mid-turn alongside a tool result rather than as a new turn, so the guard never sees them. A turn that opens with Can you ...? arms the capability-question rule, and a subsequent explicit Fix it all please cannot lift it. omind guard explain returns ALLOW for the same command, confirming the block is turn-level classification rather than command shape, and omind guard pause does not help because the rule is enforced as a hard block.

The user-visible result is the failure mode the guard is otherwise good at preventing: the agent stops and asks the user to re-issue an instruction they already gave.

Observed on macOS 25.5.0, harness=claude, semantic relevance off (keyword path).

Proudly Made in Nebraska. Go Big Red! 🌽 https://xkcd.com/2347/

…290)

Observed 2026-08-28. A turn opened with "Can you also make it save its rules
somewhere...", which correctly armed the capability-question rule. The user
then sent "Fix it all please" mid-turn — Claude Code delivers those alongside
a tool result rather than as a new turn — and the guard, which classifies from
the opening message only, kept blocking. `guard explain` returned ALLOW for the
same command, and `guard pause` could not help because the rule is enforced as
a hard block.

Net effect is the failure mode the guard otherwise prevents: the agent has to
hand the user homework, asking them to re-issue an instruction already given.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VmauK5UCYGRYyXoQ36FQ3S
@coderabbitai

coderabbitai Bot commented Aug 29, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 7323136e-7311-43de-96f9-56ad15433007


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@CryptoJones

Copy link
Copy Markdown
Owner Author

Closing as superseded — the entry this PR adds is already on main.

#290 (mid-turn user messages invisible to the authorization classifier) is in the Open section, and as of #330 it is nested under tracking issue #329 — the guard's blind spots, alongside #296. The GitHub sub-issue hierarchy and BACKLOG.md now agree on it.

This branch also predates the 2026-09-08 reconcile (#320), which removed the ### From the 2026-08-27 multi-agent review section this diff anchors against, so it would not apply cleanly regardless.

No content lost. Branch deleted.

@CryptoJones CryptoJones closed this Sep 9, 2026
@CryptoJones
CryptoJones deleted the fix/guard-midturn-authorization branch September 9, 2026 18:33
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