Skip to content

refactor: retire compliance_markers telemetry for claude sessions - #170

Merged
STRML merged 2 commits into
mainfrom
retire-claude-compliance-markers
Oct 30, 2026
Merged

STRML merged 2 commits into
mainfrom
retire-claude-compliance-markers

Conversation

@STRML

@STRML STRML commented Oct 30, 2026

Copy link
Copy Markdown
Owner

Retires compliance_markers for claude sessions. omp dropped it on 2026-08-08 and this side never followed (docs/plans/2026-10-03-omp-adapter.md: "ported for omp, not for claude").

The counter read 0 for all four markers on 2026-10-29 across a day with ~22 retry loops. The agents were following the rule: a rush-ops session "read the failure-discipline leaf and the marker script instead of retrying" and switched gates, then never typed the RETRY-BUDGET: line. So an 80-minute GUI-input loop and a clean session produced the same zero. Sam picked option B in the 2026-10-29 triage: drop the telemetry and judge retry discipline from the transcripts.

File Change
bin/session-stats.sh stops computing compliance_markers (and the $assistant_text it alone used)
prompts/SESSION_TRIAGE.md no copy-from-stats, no schema field, tool_loop severity judged on the transcript (same wording as adapters/omp/triage.md)
prompts/PROMPT.md ignore the field, no marker line in the snapshot, no marker-present/absent split
tests/run-all.sh sidecar key list drops the field; the marker fixture now asserts the field is absent

Older findings JSONs keep the field and both prompts say to leave it unread. The failure-discipline.md rule itself is unchanged.

Test plan

  • tests/run-parallel.sh 4: 3097 passed, 4 failed. The four are in review-cmux (host has cmux), adapter-claude (ps unavailable in the sandbox) and the prune-worktrees watchdog timing. None of those suites touch a changed file.
  • Red then green on the new assertion: session-stats.sh emitted compliance_markers before, not after.
  • CI on HEAD
  • /debate:run tight (repo rule for prompt and bin/ changes)

STRML added 2 commits October 30, 2026 10:15
The counter read 0 for RETRY-BUDGET, FETCH-PIVOT, DELEGATED and DIRECT-OK on 2026-10-29 across ~22 retry loops, while agents were following the rules: they made the pivot and skipped typing the line. An 80-minute loop and a clean session produced the same zero. omp retired the field on 2026-08-08; claude never followed.

session-stats.sh stops computing it, SESSION_TRIAGE.md judges tool_loop severity on the transcript (same wording as the omp addendum), PROMPT.md ignores the field. Older findings keep it unread.
… and update the plan row

Auditor seat (debate tight round 1, P2): a catch-up rebuild reuses findings JSONs triaged under the old rule, where a loop with no RETRY-BUDGET line was the serious finding. L2 now re-judges any tool_loop whose what cites a missing marker from its evidence quote. Swept prompts/ for other marker-severity text: none.

omp-review round 1 (P3): docs/plans/2026-10-03-omp-adapter.md:145 said claude still computes the field.
@STRML STRML closed this Oct 30, 2026
@STRML STRML reopened this Oct 30, 2026
@STRML
STRML merged commit 165c8be into main Oct 30, 2026
6 checks passed
@STRML
STRML deleted the retire-claude-compliance-markers branch October 30, 2026 14:54
drycode added a commit to drycode/cc-autodream that referenced this pull request Nov 6, 2026
Bring the fork level with upstream d3ea856 (STRML#170-STRML#184) so the live
install stops drifting: L2 counts from a per-session roll-up instead
of a 200-entry sample, sandbox-shape denies are counted rather than
scored as friction, changelog caches fetch anonymously and warn when
every source fails, and OMP skill reads count as invocations. The
tree is identical to upstream; the fork carries no content of its
own. install.sh was re-run to link the new rollup.sh helper.

Solves: fork 9 commits behind upstream
Tests: tests/run-parallel.sh 3241 pass; same 7 fail on 599e3ca (TMPDIR //)
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