Repository navigation
Merged
Conversation
… from specs/t27mcp/spec_brief.t27 Closes gHashTag/t27#10090. Part of gHashTag/t27#10066. Also fixes the stale backendFailures smoke check (it summed a per-backend count against a spec count). Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Summary by t27-beesThe PR makes Worth a look
Read: 8 files, +95 -43, the whole diff. Head |
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
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.
Closes gHashTag/t27#10090
Part of the epic gHashTag/t27#10066 (learnability: fewer tokens and round trips for cheap models). Stacked on #11 (
t27-examples); the spec is gHashTag/t27#10092.What
t27_specis brief by default: health, why it fails, loss, type-check errors, failing backends, size, summary, tags.fields: truereturns the whole manifest record as before. Forspecs/demos/hello_world.t27that is 619 bytes instead of 1330.t27_specsrows keep the short list they already had; it now comes from the spec.brief()andfailureReasons()are gone fromserver.mjs(35 lines out, 5 in). The field lists, the reason decision and the reason templates live inspecs/t27mcp/spec_brief.t27, run aswasm/spec_brief.wasmwithwasm/spec_brief.mjs; commands and sha256 are inwasm/BUILD. Built on the Railway lab.brief.mjs(19 lines) is glue: it passes the manifest counters to the wasm, copies the listed keys and fills the{}of the templates.smoke.mjs: the stalebackendFailurescheck is fixed (see below) and two checks are added.Hand-written JavaScript added:
brief.mjs19,server.mjs5,smoke.mjs9 = 33 lines (cap: 40 per file, 80 per PR).server.mjsshrinks by 30 net.The smoke check that failed on #7
backendFailures tally matches the specs blaming a backendcompared the sum of a per-backend count (737) with a count of specs (113). Not a server bug: the manifest'sbackendFailurescounts specs per backend (c, js, rust, ts, verilog_hir, zig: 104 each; verilog: 113), and the sum of those is not a number of specs. The check now compares each backend's count witht27_specs {failedBackend}matched, and all seven agree.Evidence
npm run smoke: all checks pass, includingt27_spec is brief by default,every failing row carries a reason from the spec(113 rows, no unfilled{}) and the per-backend check.🤖 Generated with Claude Code