Repository navigation
Board token: agents need no board ID, changes are attributed to the analyst - #5
Merged
Merged
Conversation
…nalyst Agents connected with the token in the X-FactGraph-Token header were asked for the board ID and the token: the server instructions said so, and every agent tool required board_id. The token already names the board (each browser tab makes its own token for each board), so the server now takes the board from it. - Agent tools no longer show board_id; one passed anyway (older prompts) is accepted and must match the token. The full REST-parity profile shows it as optional. A token without an open tab gets a clear 409. - "Session token" is now "board token" in the UI, docs and MCP instructions, and no longer shares a name with Entra session tokens (uti) in the impact analysis. The MCP parameter is board_token; session_token is still accepted but not shown. - Changes an agent makes with an analyst's token are recorded as that analyst's agent: "MCP (Gregor)", "REST (Gregor)" or the agent's own name, "Claude (Gregor)". Notifications and evidence provenance read "Agent via MCP (Gregor)". Trust rules still depend only on the channel. - The connect dialog explains the token and no longer puts a board ID into the suggested prompt. Agent tool budget unchanged (about 9,700 tokens). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Agents connected with the token in the
X-FactGraph-Tokenheader still asked the analyst for the board ID and the token. The server instructions told them to ("the analyst gives you board ID and token"), and every agent tool hadboard_idas a required parameter. The token already names the board, though: each browser tab creates its own token for each board it opens.What changed
board_of_session).board_id. One passed anyway (older prompts, the old suggested first message) is still accepted and must match the token, as before.board_idas optional, with a description.uti) in the attack-impact analysis; those texts are unchanged.board_token.session_tokenis still accepted but hidden from the schema. The header name is unchanged.MCP (Gregor),REST (Gregor)or the agent's own name (Claude (Gregor)). Agent undo/redo are labelled the same way.For reviewers
Import, which the auto-confirm rule relies on.board_id/session_tokenonly changes the advertised tool schema. FastMCP validates arguments against the function, which still accepts both.Tests
board_id, a call that still passes it, a token without an open board, and that agent tool schemas show noboard_id. Contract tests check the instructions and the parity profile.onBehalf,agentText).MCP (Test analyst)/REST (Test analyst), and the Connect dialog test checks that the prompt has no board ID.🤖 Generated with Claude Code