Skip to content

feat(skills): baseline-source ladder - greenfield and recipe-confirmed baselines - #13621

Closed
BenHamm wants to merge 5 commits into
optimization-pack-v2from
bhamm/greenfield-baseline-ladder
Closed

feat(skills): baseline-source ladder - greenfield and recipe-confirmed baselines#13621
BenHamm wants to merge 5 commits into
optimization-pack-v2from
bhamm/greenfield-baseline-ladder

Conversation

@BenHamm

@BenHamm BenHamm commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

What

Greenfield engagements, properly designed (the fast-follow cut from #13557): the interview may now produce the baseline when the user has none, with the user's explicit confirmation as the invariant at every rung:

  1. User provides a DGD (origin: user) - unchanged.
  2. Exact/close catalog recipe proposed with an explicit adaptation diff (origin: recipe-confirmed).
  3. Nothing close: new author-baseline-dgd skill drafts one from the interview facts and the model-sizing/knob-tuning guides, presented with a per-decision evidence table (origin: agent-authored).

Selection and authoring happen ONLY at interview time; the loop still never selects. Provenance is recorded (deployment.origin/origin_source; inputs/baseline-evidence.md carries the proposal, evidence, and confirmation). Non-user origins calibrate downstream: iteration 0 is pure characterization, and the generator/challenger treat the baseline itself as a hypothesis.

Validation

  • Two adversarial verification rounds over the contract corpus (9 findings first round, 1 second round - all fixed; 26 skills validate).
  • Full greenfield clean room on live H200s: scripted SA persona requested Qwen3-4B (no catalog recipe exists). The agent correctly landed rung 3, authored a baseline with real sizing arithmetic (min_tp=1, headroom 0.94) and evidence-backed backend choice, waited for confirmation, deployed via the hardened deploy blocks (one compat patch, ledger-recorded, canonical baseline untouched), passed smoke, and characterized: TTFT p95 797.8ms vs the 2s SLO, zero errors in 400 requests, ~0.9 of 4 granted GPU-hours consumed and reported. One finding (headless harnesses must poll, not park) fixed in the last commit.

Sequencing

Base is optimization-pack-v2; after #13557 merges this retargets to main.

Reviewer starting points

  • .agents/skills/author-baseline-dgd/SKILL.md (new)
  • agents/user-interviewer/AGENTS.md (the ladder)
  • agent-docs/rules/execution/user-workload.md (origin schema)

🤖 Generated with Claude Code


Open in Devin Review

BenHamm and others added 2 commits August 20, 2026 16:44
…d baselines

The interview may now produce the baseline when the user has none, with
the user's explicit confirmation as the invariant at every rung:

1. user provides a DGD (origin: user)
2. exact/close catalog recipe proposed with an explicit adaptation diff
   (origin: recipe-confirmed)
3. no close recipe: new author-baseline-dgd skill drafts one from the
   interview facts and the model-sizing/knob-tuning guides, presented
   with a per-decision evidence table (origin: agent-authored)

Selection and authoring happen ONLY at interview time; the loop still
never selects. Provenance recorded as deployment.origin/origin_source;
inputs/baseline-evidence.md carries the proposal, evidence table, and
confirmation for rungs 2-3. Non-user origins calibrate downstream:
iteration 0 is pure characterization, and the generator/challenger
treat the baseline itself as a hypothesis (topology families start
untested). The incompatible-baseline graceful end now offers the ladder
for the follow-on engagement.

Gate: two adversarial verification rounds (9 findings first round, 1
second round - all fixed); 26 skills validate.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…tions

Found in the greenfield validation room: a headless -p agent parked the
engagement on a background-job notification that can never arrive in a
single-shot harness. The long-running-runs guidance now distinguishes
harness types explicitly.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@BenHamm
BenHamm requested review from a team as code owners August 21, 2026 00:23
@github-actions github-actions Bot added feat documentation Improvements or additions to documentation labels Aug 21, 2026

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Review feedback on #13557 (landed here to avoid dismissing its fresh
approvals): SKILL.md files self-discover, but the orchestration guide
with the sequencing and stopping rules does not - the dispatch section
now requires the top-level agent to read it first.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
BenHamm and others added 2 commits August 20, 2026 20:49
…ncy-grid

The rule said 'at most 4x the concurrency' while its example said c=4
means at most 4 requests (1x). An agent following the example produces
4-request measurements that cannot support the noise-floor rules.
Found by automated review on #13557; landed here to preserve that PR's
approvals.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@BenHamm

BenHamm commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

This PR was auto-closed when its base branch was deleted at the #13557 merge. Continued as #13625 (same commits, rebased onto main, plus the follow-up fixes from the review threads).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation feat size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant