Skip to content

skill: a first-error count is not an unblocking count - #3211

Merged
gHashTag merged 4 commits into
masterfrom
first-error-is-not-a-yield
Sep 5, 2026
Merged

gHashTag merged 4 commits into
masterfrom
first-error-is-not-a-yield

Conversation

@gHashTag

@gHashTag gHashTag commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

Recorded from #3208, where the two numbers differed by a factor of six and
quoting the larger one would have been wrong.

The ungated serde derive was the first rustc error in 84 specs, and the form
discriminated perfectly — 0 of the 224 passing files, 84 of the 357 failing ones.
Measured by name over all 650 specs after the fix, rustc acceptance moved 224 →
237
: +13 unblocked, 0 regressions, and 0 specs still failing on serde.

Errors queue. 84 is how many specs the cause stops first; 13 is how many the fix
carries all the way to acceptance; the other 71 report whatever stood behind it. A
class can be the largest single cause in a column and still unblock a small fraction
of it — that is what a backlog with depth looks like, not a failure of the fix.

Three separate claims, none implying another, so all three belong in a report:

  • the first-error count proves the cause is large
  • the by-name before/after proves what the fix yields — diffing by spec name is
    what stops a gain and a regression cancelling inside a total
  • a count of specs still failing on that cause proves it was removed rather than
    reduced

Skill §560 and a docs/now entry. No code.

Refs #3208

From #3208. The ungated serde derive was the FIRST rustc error in 84 specs and
discriminated perfectly -- 0 of 224 passing files, 84 of 357 failing. Measured by
name after the fix, rustc acceptance moved 224 -> 237: +13 unblocked, 0
regressions, and 0 specs still failing on serde.

Errors queue. 84 is how many specs the cause stops first; 13 is how many the fix
carries to acceptance. Both are true and neither implies the other, so a report
needs the first-error count, the by-name before/after, and a count of specs still
failing on the cause -- three separate claims.

Refs #3208

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-09-04 23:57:25 UTC

Summary

Status Count
Total Open PRs 14
PRs with Failing Checks 11
PRs with All Checks Green 3
READY 0
FAILING 11
PENDING 0

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=9b8875f1c9d4 != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

…a-yield

# Conflicts:
#	.claude/skills/ci-gates/SKILL.md
@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-09-05 00:04:07 UTC

Summary

Status Count
Total Open PRs 13
PRs with Failing Checks 11
PRs with All Checks Green 2
READY 1
FAILING 11
PENDING 0

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=9b8875f1c9d4 != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

…a-yield

# Conflicts:
#	.claude/skills/ci-gates/SKILL.md
@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-09-05 00:33:04 UTC

Summary

Status Count
Total Open PRs 14
PRs with Failing Checks 11
PRs with All Checks Green 3
READY 0
FAILING 11
PENDING 0

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=3c8c4b4c629b != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-09-05 00:39:44 UTC

Summary

Status Count
Total Open PRs 13
PRs with Failing Checks 11
PRs with All Checks Green 2
READY 1
FAILING 11
PENDING 0

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=7f1f58f33681 != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

@gHashTag
gHashTag merged commit bcd4ecf into master Sep 5, 2026
28 checks passed
@gHashTag
gHashTag deleted the first-error-is-not-a-yield branch October 26, 2026 06:06
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.

1 participant