weekly sync — 2026-05-04#1
Open
norbert-pap wants to merge 1 commit into
Open
Conversation
Source: garrytan/gstack PR #1313 → eng-review/SKILL.md Hardens all 5 STOP gates (Step 0 complexity check + Architecture + Code Quality + Test Review + Performance) with the office-hours anti-rationalization pattern: - "AskUserQuestion is a tool_use, not prose — call the tool directly" - Named blocked next steps: do NOT proceed to next section or edit the plan file until user responds - Anti-rationalization clause: loading AskUserQuestion via ToolSearch then writing prose is the failure mode this gate exists to prevent - Architecture gate only: "obvious fix still needs explicit approval" Also updates completion summary: Step 0 label now shows "scope accepted as-is / scope reduced" and adds Lake Score metric (tracks completeness-principle recommendations). ExitPlanMode / Decisions-to-confirm fallback dropped (not in claudebert). Template tokens dropped.
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.
Sources checked (6)
Applied changes (1)
eng-review/SKILL.md← garrytan/gstack PR #1313v1.26.2.0 fix: plan-eng-review STOP gates always fire + report-at-bottom contract
Hardened all 5 STOP gates (Step 0 complexity check + Architecture + Code Quality + Test Review + Performance) with the anti-rationalization pattern from upstream:
AskUserQuestion is a tool_use, not prose — call the tool directlyadded to each gate headerAlso updated the completion summary: Step 0 label now reads
scope accepted as-is / scope reduced per recommendation, and a Lake Score metric was added (X/Y recommendations chose complete option) to track adherence to the Completeness Principle already in the skill.Porting notes:
ExitPlanMode/Decisions to confirmfallback dropped (claudebert has no plan-mode preamble).{{TOOL_RESOLUTION}}template token dropped. Core safety language kept verbatim.Deferred — noteworthy but not applied (3)
garrytan/gstack PR #1233 — browser-skills runtime v1 (2026-04-29)
53-file PR adding a new domain-skill pattern (
browser-skills/hackernews-frontpage/SKILL.md) and expanding the browse binary with CDP bridge, domain-skill commands, skill tokens, and tab isolation. No changes tobrowse/SKILL.md.tmpl— the skill documentation was not updated to reflect new commands. Deferred: would need a separate pass to determine which new capabilities (if any) should be surfaced in claudebert'sbrowse/SKILL.md; not a clean text cherry-pick.addyosmani/agent-skills PR #101 — JSON-escape hook content in session-start hook
Fixes
hooks/session-start.shto usejq --argso SKILL.md with quotes/newlines produces valid JSON. Hook infrastructure change; no skill content. Dropped (claudebert session-start hook is independent).anthropics/skills PR #1081 — Remove non-existent
purposefield from Files API examplesRemoves an incorrect field from API documentation examples. No skill content change. Dropped.
Generated by Claude Code