Repository navigation
chore(dz): allow the tracker reads the briefs already name - #823
developerz-ai[bot] merged 1 commit into
Conversation
.claude/settings.json permitted `gh pr view`, `gh run list` and `gh run view` but not `gh issue list` / `gh pr list`. A box taking the release-due brief hit its own "never file a second ask" rule and could not run the read that rule needs, so it stalled or filed a duplicate. Grant both, and name the query at .dz/briefs/release-due.md:34 so the grant matches a command the brief actually runs. .dz/briefs/health.md already used both commands the same way. Dz-Task-Id: tsk_ed59b9e5c6829472e14a96a722ee559a
|
Add gh issue list (and gh pr list) to .claude/settings.json allow list
Work landed on #823 — watching it toward merge. 108h19m · 46 tool calls · 3 messages Sessions: coding run 🤖 developerz.ai, an automated agent, not a human — hv-dedicated-dev-marshall (developerz-ai[bot]) · model |
✅ Reviewed — nothing blocking0 actionable comment(s) · grounded on your code ⏱ 7m 46s wall clock · 🤖 developerz.ai — automated review, running on your model and your box. |
There was a problem hiding this comment.
Review summary — 2 file(s), 0 finding(s).
Completed review [bot]: adds tracker-read permissions and an explicit release-ask lookup. The supplied diff preserves the existing permissions and hook, keeps valid JSON, and shows no actionable defects.
Config notes
- test files were not shown to the reviewer — this diff touched no path recognised as a test, so no assertion was available to read the change against
🤖 developerz.ai review — automated, running on your model and your box. What is this?
Reviewed by router/gpt-6.1-sol on box hv-dedicated-ci-1 — 14.7k in / 206 out over 1 call; cost unknown: no price on record for router/gpt-6.1-sol.
What
.claude/settings.json permitted
gh pr view,gh run listandgh run viewbut notgh issue list/gh pr list. A box taking therelease-due brief hit its own "never file a second ask" rule and could
not run the read that rule needs, so it stalled or filed a duplicate.
Grant both, and name the query at .dz/briefs/release-due.md:34 so the
grant matches a command the brief actually runs. .dz/briefs/health.md
already used both commands the same way.
Acceptance
gh issue listandgh pr listat lines 17-18 are now permitted.x newwrites into a new app.Changes
.claude/settings.json(edit) — Add "Bash(gh issue list:)" and "Bash(gh pr list:)" to permissions.allow, in the gh group after line 19's "Bash(gh run view:*)", keeping valid JSON, 2-space indent and a trailing-comma-free last entry..dz/briefs/release-due.md(edit) — Rewrite the line-34 rule "Never file a second ask while one titledRelease …is open" so it names the exact read the box must run — a gh issue list query over open issue titles forRelease <next>— so the new grant matches a command the brief actually runs. Match the table style already used in .dz/briefs/health.md:17-18.Verification
bun run verifypassed (exit 0) in 1h 06m 04s.Notes
Premises of the request the team disproved:
x newscaffold, asserted by scaffold-claude.test.ts:38,131). Out of scope, but an editor grepping "settings.json" will find it and must not edit it.🤖 Opened by developerz.ai for task
tsk_ed59b9e5c6829472e14a96a722ee559aon developerz-ai/ultimate.Raised automatically by the
raw_task_triagelane.