Conversation
ColtenOuO
reviewed
Oct 3, 2026
A report could read "behavioral (skipped)" beside STAR scores and a summary of the STAR answer, because the interviewer can write its own round-start event, ask a behavioral question at minute 7, and record STAR evidence, while only the platform's transition sets the round status. The evidence tool now refuses every STAR phase until that transition and tells the interviewer to return to coding, and the live prompt says only the platform sends a stage direction. The report brief says whether the round opened, never opened, or does not exist; an opened round is marked in the report transcript, so an answer given out of turn before the mark is not assessed. For an unopened round, validation refuses a STAR plan item, so the repair loop rewrites it from the coding round rather than the report being trimmed below the two improvements each list must hold, and the server clears the STAR scores of the report it accepts, which costs no repair. A coding-only interview no longer gets STAR skip rows, which with the ledger full also evicted a coding observation. Bundle 27 carries live prompt 19 and report prompt 16. Closes #211
The behavioral round mark and the helper that inserts it were re-exported as public agent API, though only the report builder and its unit tests use them. Both are now crate-private, the mark visible to tests only, so a prompt-format detail is not a surface a dependent can come to rely on.
ColtenOuO
approved these changes
Oct 3, 2026
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.
A report could read
behavioral (skipped)beside STAR scores and a summary of the STAR answer. The interviewer can write its own[SYSTEM EVENT] ... Begin STAR behavioral reserve, ask a behavioral question at minute 7, and record STAR evidence, while only the platform'sround_transitionsets the round status the report reads. This takes the two fixes the issue thread asked for:record_framework_evidencerefuses every STAR phase until the platform opens the round (platform-owned skips at the five-minute warning and the end still close it), and the report can no longer pair a skipped round with STAR content.On the report side, the brief now says whether the behavioral round opened, never opened, or does not exist in this interview, and tells the reviewer an out-of-turn exchange is not evidence for any field. Validation refuses a STAR score or STAR plan item for an unopened round, so the existing repair loop rewrites them from the coding round; trimming them after validation instead would drop the matching feedback improvements and could leave a list below the two items the schema requires. The live prompt now says only the platform sends a stage direction. A coding-only interview no longer gets STAR skip rows, which with the evidence ledger full also evicted a coding observation. Bundle 27 carries live prompt 19 and report prompt 16; the rubric and report schema are unchanged.
Verified with
./scripts/test.sh, which passes locally. The browser lanes skipped for lack of Chromium and Playwright on this machine, as didcargo-auditandshellcheck. New tests cover the validator refusing and accepting by round state, the repair loop rewriting and then running out on STAR content, the frozen assessment and real prompt builder carrying the round state, a STAR answer after the five-minute warning, and a coding-only interview getting no skips.Left out: the header minutes are the planned budgets rather than time spent, and whether a coding round that finishes early may open the behavioral reserve before the coding budget elapses. Both were set aside in the issue as separate questions.
Closes #211
Summary by cubic
Stops STAR evidence from being recorded before the platform's round transition opens the behavioral round, so a report can no longer show STAR scores and a summary of the STAR answer beside
behavioral (skipped).record_framework_evidencenow refuses every STAR phase until the round opens and tells the interviewer to return to coding; platform-owned skips at the five-minute warning and the end still close an unopened round.[SYSTEM EVENT], and a coding-only interview no longer gets STAR skip rows, which with a full ledger also evicted a coding observation.Written for commit 68ff727. Summary will update on new commits.