Skip to content

weekly sync — 2026-05-25#5

Open
norbert-pap wants to merge 1 commit into
mainfrom
sync/2026-05-25
Open

weekly sync — 2026-05-25#5
norbert-pap wants to merge 1 commit into
mainfrom
sync/2026-05-25

Conversation

@norbert-pap
Copy link
Copy Markdown
Owner

Applied changes

1. Pre-emit verification gate → code-review + eng-review

Source: garrytan/gstack#1642 (v1.43.2.0 — "fix wave: post-Daegu paper-cut")
Why cherry-picked: Kills an entire class of false-positive findings ("field doesn't exist on model X") by requiring every finding to cite a specific code line before it's surfaced. Self-contained rule with no gstack-internal dependencies.

code-review/SKILL.md — Added ### Pre-emit verification gate section inside Phase 2 (Five-Axis Review), before Axis 1. Rule: quote file:line + verbatim text that motivates the finding; unverified findings are downgraded to Nit/FYI (suppressed from Critical/High). Includes framework-meta nudge for ORM/metaclass symbols (Django Meta, Rails has_many, SQLAlchemy relationship, TypeORM decorators, Prisma generated client).

eng-review/SKILL.md — Added ## Pre-emit verification gate section before ## CRITICAL RULE. Same quotation requirement; unverified findings are dropped silently rather than presented via AskUserQuestion.

Porting notes: Dropped the internal #1539 issue reference. Adapted the confidence-score suppression (gstack uses 1–10 numeric scale) to claudebert's severity labels (Critical/High/Nit/FYI for code-review; drop-silently for eng-review). Core rules and framework-meta nudge kept verbatim.


Deferred / noteworthy (not applied)

anthropic-skills PR #1164 — claude-api skill updates

New managed-agents docs: managed-agents-self-hosted-sandboxes.md, EnvironmentWorker class, mid-session sessions.update() for tool/MCP overrides, GET /v1/environments/{id}/work/stats. No mapping in claudebert's pipeline (no claude-api skill exists). Would require creating a new skill — out of scope for a cherry-pick sync.

gstack PR #1678 — v1.44.0.0 long-lived sidebar

PTY keepalive (25s ping/pong), dual session/auth token model, ring-buffer scrollback replay, terminal-agent watchdog, --supervise CLI flag. Pure gstack PTY server infrastructure — no SKILL.md changes. Not applicable.

gstack PR #1645 — v1.43.3.0 fix(browse): headed-mode idle timer

browse/src/server.ts bug fix: four lifecycle handlers were reading a stale module-level BrowserManager instead of the embedder-supplied one. Implementation fix only — no SKILL.md changes. Not applicable.

VoltAgent/awesome-agent-skills — new skill additions

Three new skills added this week: digital-marketing-pro, career-ops, dembrandt-skills. Repo has 22,976 stars (well above 50-star threshold). None fill claudebert pipeline gaps (review, qa, design-review, browse, code-review, eng-review, prod-review, design-doc).


Sources checked (6)

id repo commits since 2026-05-18 action
gstack garrytan/gstack 3 1 cherry-pick applied
agent-skills addyosmani/agent-skills 7 none relevant to pipeline
anthropic-skills anthropics/skills 1 deferred (no skill mapping)
awesome-agent-skills VoltAgent/awesome-agent-skills 8 deferred (no pipeline gap filled)
awesome-claude-toolkit rohitg00/awesome-claude-code-toolkit 0
designpowers Owl-Listener/designpowers 0

Generated by Claude Code

Port the pre-emit verification gate from garrytan/gstack#1642 (v1.43.2.0)
into code-review and eng-review skills. Every finding must now quote the
specific motivating code line (file:line + verbatim text) before being
promoted; unverified findings are downgraded to Nit/FYI (code-review) or
dropped silently (eng-review). Includes framework-meta nudge for ORM and
metaclass-generated symbols.

Also adds .sync-state.json recording this sync run.
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.

2 participants