From c716935facc7aba45f81ead66353bc7b0441539f Mon Sep 17 00:00:00 2001 From: Aaron Stainback Date: Wed, 29 Apr 2026 02:08:51 -0400 Subject: [PATCH 1/6] absorb: multi-AI feedback packet (Deepseek + Amara) + 4 backlog rows + tick 0558Z shard MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The human maintainer forwarded a multi-AI synthesis packet during autonomous-loop tick 05:58Z: - Deepseek's reassessment + 5-point pushback after correcting earlier-incorrect search results - Amara's filter-to-actionables (6 bounded items, with "rerun is incident recovery; retry/cache is substrate improvement" elevated as the load-bearing line) - reference to an older Gemini log on tele+port+leap operational resonance (already canonical at memory/feedback_operational_resonance_*.md; not re-absorbed) Verbatim absorb (per the channel-verbatim-preservation rule) landed at: docs/research/multi-ai-feedback-2026-04-29-deepseek-amara-on-loop-state.md with §33 archive header (Scope / Attribution / Operational status / Non-fusion disclaimer). Four small P3 backlog rows file the bounded actionables (per the maintainer's existing narrowing on task #309 — research- grade only, no broad new substrate PRs): B-0098 — tick-ordinal-continuity lint (or remove ordinals from shards entirely; computed > narrated) B-0099 — PR-count claims as derived metrics, not narrated prose B-0100 — pure-wait tick backpressure / quiescence rule B-0101 — small 5-bucket reviewer-artifact classification table The 5th actionable (external-dep retry/cache) is already addressed by PR #804 (durable-retry fix landed alongside the "rerun-is-recovery / retry-is-substrate-improvement" rule). The 6th actionable (evidence-claim language tightening) is operational discipline, captured in the tick shard observation column. Tick shard at docs/hygiene-history/ticks/2026/04/29/0558Z.md captures the work-stream summary. Pattern: ONE consolidated PR for the absorb bundle (research note + 4 backlog rows + tick shard) rather than 6 separate PRs, honoring the maintainer's "don't open a bunch of new PRs" narrowing while still preserving the verbatim record durably. Co-Authored-By: Claude Opus 4.7 --- ...tick-ordinal-continuity-lint-2026-04-29.md | 73 ++++++++ ...ount-projection-not-narrated-2026-04-29.md | 50 +++++ ...backpressure-quiescence-rule-2026-04-29.md | 88 +++++++++ ...t-classification-small-table-2026-04-29.md | 93 ++++++++++ .../hygiene-history/ticks/2026/04/29/0558Z.md | 1 + ...2026-04-29-deepseek-amara-on-loop-state.md | 172 ++++++++++++++++++ 6 files changed, 477 insertions(+) create mode 100644 docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md create mode 100644 docs/backlog/P3/B-0099-pr-count-projection-not-narrated-2026-04-29.md create mode 100644 docs/backlog/P3/B-0100-pure-wait-tick-backpressure-quiescence-rule-2026-04-29.md create mode 100644 docs/backlog/P3/B-0101-reviewer-artifact-classification-small-table-2026-04-29.md create mode 100644 docs/hygiene-history/ticks/2026/04/29/0558Z.md create mode 100644 docs/research/multi-ai-feedback-2026-04-29-deepseek-amara-on-loop-state.md diff --git a/docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md b/docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md new file mode 100644 index 000000000..2119de888 --- /dev/null +++ b/docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md @@ -0,0 +1,73 @@ +--- +id: B-0098 +priority: P3 +status: open +title: Tick-ordinal-continuity lint — ordinals are computed, not narrated +tier: research-grade +effort: S +ask: Multi-AI synthesis packet 2026-04-29 (Deepseek + Amara filter) +created: 2026-04-29 +last_updated: 2026-04-29 +composes_with: [B-0099, B-0100, B-0101] +tags: [ci-lint, tick-history, derived-metadata, manual-drift-class, mechanical-guard] +--- + +# Tick-ordinal-continuity lint + +Hand-authored tick-ordinal words ("twenty-second", "twenty-third", +etc.) in `docs/hygiene-history/ticks/YYYY/MM/DD/*.md` shards drift +under edit pressure. The 2026-04-29 ordinal-cascade incident showed +the failure mode: a Copilot review thread cited a snippet, my "fix" +trusted the cited context without re-reading the full file +sequence, and introduced a duplicate ordinal. Recovery worked +(read all shards → revert → re-verify), but the underlying class +is *manual metadata drift* — the same class as the bare-`main` +ambiguity rule. + +## Two viable resolutions (pick one or compose) + +### Option A — Lint that verifies claimed ordinals against file order + +```bash +# Pseudocode +for file in $(ls -1 docs/hygiene-history/ticks/YYYY/MM/DD/*.md | sort); do + claimed_ordinal=$(grep -oE '\b(first|second|...|thirtieth|...)\b' "$file" | head -1) + expected_ordinal=$(compute_from_file_position) + [[ "$claimed_ordinal" == "$expected_ordinal" ]] || warn "$file ordinal mismatch" +done +``` + +Pros: keeps the prose readable. +Cons: still depends on prose, just with a guard. + +### Option B — Drop ordinal words from shards entirely; compute in projection + +Shards become pure event data (timestamp, model, cron-id, body, +PR refs, observation). The "twenty-second tick" framing lives only +in a generated read-model (`docs/hygiene-history/projection/...`) +that derives the ordinal from file position. + +Pros: eliminates the failure class entirely. Aligns with the DBSP/ +Z-set pattern Amara cited (event = stored, projection = derived). +Cons: requires updating the existing shard schema + tooling. + +## Recommendation + +Lean toward Option B (the cleaner fix) when next active work +window opens. Option A as an intermediate guard if prose has to +stay for a few more rounds. + +## Composes with + +- `memory/feedback_bare_main_ambiguity_automation_discipline_explicit_refs_required_amara_2026_04_29.md` + — same class (computed > narrated metadata). +- B-0099 (PR-count projection) — sibling action item from the same + packet. + +## Why P3 (research-grade, not blocking) + +The ordinal drift was caught and corrected within 2-3 ticks via the +existing review pipeline. The mechanical guard would prevent +future occurrences but is not blocking; opening it as P0/P1 would +violate the maintainer's narrowing on multi-AI-synthesis-packet +items. Promote when work-bandwidth allows. diff --git a/docs/backlog/P3/B-0099-pr-count-projection-not-narrated-2026-04-29.md b/docs/backlog/P3/B-0099-pr-count-projection-not-narrated-2026-04-29.md new file mode 100644 index 000000000..796a3cead --- /dev/null +++ b/docs/backlog/P3/B-0099-pr-count-projection-not-narrated-2026-04-29.md @@ -0,0 +1,50 @@ +--- +id: B-0099 +priority: P3 +status: open +title: PR-count claims are derived metrics — compute, don't narrate +tier: research-grade +effort: S +ask: Multi-AI synthesis packet 2026-04-29 (Deepseek + Amara filter) +created: 2026-04-29 +last_updated: 2026-04-29 +composes_with: [B-0098, B-0100, B-0101] +tags: [tick-history, derived-metadata, manual-drift-class, mechanical-guard, projection] +--- + +# PR-count claims are computed, not remembered + +Hand-authored "session PR totals" in tick-history shards (e.g. +"30 PRs total this session arc") drift across shards. The +2026-04-29 session arc had to soften several claims to "approximate" +because the asserted totals diverged from the actual git log. +Same failure class as the tick-ordinal drift (B-0098): derived +metadata authored as prose. + +## Resolution + +When a tick shard needs to cite a session PR total: + +1. **Don't write it as authoritative prose.** No "this is the + thirty-first PR of the arc" style claims unless computed. + +2. **If included, cite the computation.** E.g., + `count via gh pr list --search 'merged:>=2026-04-29 author:@me' | wc -l`. + +3. **Better: move totals to a generated projection.** A small + script that walks `docs/hygiene-history/ticks/YYYY/MM/DD/*.md` + + `gh` API + `git log` and produces a session-summary file + eliminates the manual drift class entirely. + +## Composes with + +- B-0098 (tick-ordinal-continuity lint) — sibling derived-metadata + action item. +- `memory/feedback_bare_main_ambiguity_automation_discipline_explicit_refs_required_amara_2026_04_29.md` + — same computed-vs-narrated discipline. + +## Why P3 + +Same reasoning as B-0098 — the drift was caught and the prose +softened. Mechanical fix is bounded but research-grade-only per +the maintainer's narrowing. diff --git a/docs/backlog/P3/B-0100-pure-wait-tick-backpressure-quiescence-rule-2026-04-29.md b/docs/backlog/P3/B-0100-pure-wait-tick-backpressure-quiescence-rule-2026-04-29.md new file mode 100644 index 000000000..6e4beb266 --- /dev/null +++ b/docs/backlog/P3/B-0100-pure-wait-tick-backpressure-quiescence-rule-2026-04-29.md @@ -0,0 +1,88 @@ +--- +id: B-0100 +priority: P3 +status: open +title: Pure-wait tick backpressure / quiescence rule for autonomous-loop tick shards +tier: research-grade +effort: M +ask: Multi-AI synthesis packet 2026-04-29 (Deepseek + Amara filter) +created: 2026-04-29 +last_updated: 2026-04-29 +composes_with: [B-0098, B-0099, B-0101] +tags: [autonomous-loop, tick-history, backpressure, quiescence, repo-hygiene, soulfile] +--- + +# Pure-wait tick backpressure / quiescence rule + +Right now every `<>` cron fire produces a +tick-history shard PR, even if the only tick activity was "queue +checked, nothing to merge, CI in progress, no events." Over a +long-enough operational drain the repo accumulates thousands of +near-identical "pure-wait" shards. + +The packet's strategic point: there's currently no termination +condition. + +## Two-tier proposal + +### Tier 1 — Backpressure (immediate) + +```text +If open tick-history PRs >= 3 +and the tick has no substantive event +and only CI is pending, +do not open another pure-wait shard. +Batch or skip until one lands. +``` + +Implementable as a check in the tick-close procedure: count open +`tick-history/*` PRs; if ≥ N and the tick body would be a +pure-wait note, skip the shard creation step. + +### Tier 2 — Quiescence (after Tier 1 lands) + +```text +After M consecutive no-op / CI-wait ticks, +enter quiescent mode: +- stop per-minute shards +- run periodic health checks (e.g., hourly) +- wake on external signal: + - new PR opened + - failing build + - maintainer input + - unresolved review thread +``` + +Implementable as a CronCreate cadence change on detection of +quiescence + a re-arm trigger. + +## Why this matters + +Per the soulfile-cleanliness rule +(`memory/feedback_repo_is_soulfile_dont_commit_raw_diagnostic_dumps_aaron_amara_2026_04_29.md`), +text compresses well in pack-delta storage so individual +near-identical shards aren't a soulfile risk. But: + +- Repo browse-ability suffers (thousands of shards = hard to + scan). +- PR-review backlog accumulates (each shard needs a CI cycle). +- The factory's signal-to-noise ratio degrades — "what changed + this hour" becomes "scroll through 60 shards." + +## Why P3 + +The loop is currently in productive operational drain — every +recent tick has had real events (PR merges, review-thread fixes, +mid-tick corrections). The quiescence rule fires only when drain +runs out. Backpressure is a slightly more proactive safeguard +but still bounded. Promote when active drain is clear. + +## Composes with + +- B-0098, B-0099, B-0101 — sibling actionables from the same + packet. +- `docs/AUTONOMOUS-LOOP.md` — current per-minute cron contract. +- `memory/feedback_never_idle_speculative_work_over_waiting.md` + — the never-idle invariant; quiescence is its dual (idle is OK + when queue is genuinely empty, but only after sustained + evidence). diff --git a/docs/backlog/P3/B-0101-reviewer-artifact-classification-small-table-2026-04-29.md b/docs/backlog/P3/B-0101-reviewer-artifact-classification-small-table-2026-04-29.md new file mode 100644 index 000000000..4a2cb1b9f --- /dev/null +++ b/docs/backlog/P3/B-0101-reviewer-artifact-classification-small-table-2026-04-29.md @@ -0,0 +1,93 @@ +--- +id: B-0101 +priority: P3 +status: open +title: Reviewer-artifact classification — small 5-bucket table (Copilot + future review agents) +tier: research-grade +effort: S +ask: Multi-AI synthesis packet 2026-04-29 (Deepseek + Amara filter) +created: 2026-04-29 +last_updated: 2026-04-29 +composes_with: [B-0098, B-0099, B-0100] +tags: [code-review, copilot, reviewer-noise, taxonomy, classification] +--- + +# Reviewer-artifact classification — small 5-bucket table + +The autonomous-loop is now effectively a two-agent loop: the +acting agent (Otto / Claude) + the reviewing agent (Copilot +pull-request-reviewer, plus increasingly cross-AI peers). The +review agent has its own failure modes that the acting agent +needs to distinguish from real findings. + +Recent recurring reviewer-failure patterns from this session: + +- `display-artifact` — Copilot hallucinated "leading space and + `1 ||`" prefix that wasn't in the actual file. +- `time-travel-review-artifact` — reviewer evaluated a PR snapshot + before cited dependency PRs merged; the reference was valid on + current main. +- `incomplete-cited-context` — the reviewer's snippet was correct + but not load-bearing for the issue; trusting it without + re-reading the full file caused an over-correction + (the 2026-04-29 ordinal-drift cascade). + +## Proposed small classification table (5 buckets, intentionally +## small per Amara's filter) + +```text +REAL_DEFECT + - actual code/doc/test issue caught by reviewer + - action: fix in current PR + +REVIEWER_SNAPSHOT_LAG + - reviewer evaluated a stale snapshot of repo state + - cited reference is now valid on current main + - action: comment with merged SHAs, resolve thread + +DISPLAY_ARTIFACT + - reviewer's quoted excerpt contains characters not in source + - benign hallucination from review-tool rendering + - action: resolve with brief explanatory comment + +INCOMPLETE_CONTEXT + - reviewer's cited snippet is correct but not load-bearing + - fix should re-read full file before applying + - action: read full source, decide if fix is real or no-op + +NEEDS_HUMAN_REVIEW + - reviewer flagged something the acting agent can't classify + - action: leave thread open, surface to maintainer +``` + +## Where this lands + +Two options: + +1. **As a memory file** — `memory/feedback_reviewer_artifact_classification_5_buckets_2026_04_29.md` — + gives future-Claude a quick decision tree. + +2. **As a section in a code-review skill** — embed in + `.claude/skills/code-reviewer/` or similar, so the + classification is invoked at decision time, not lookup + time. + +Decision deferred to whoever picks up the action item. + +## Why P3 + +The classification works informally now — most reviewer findings +get correctly classified by acting-agent judgment. Codifying it +hardens the discipline but isn't blocking. Promote when active +drain is clear. + +## Composes with + +- B-0098, B-0099, B-0100 — sibling actionables from the same + packet. +- `memory/feedback_otto_355_blocked_with_green_ci_means_investigate_review_threads_first_dont_wait_2026_04_27.md` + — the BLOCKED-with-green-CI rule that makes review-thread + triage a load-bearing tick step. +- `memory/feedback_outdated_review_threads_block_merge_resolve_explicitly_after_force_push_2026_04_27.md` + — outdated-thread resolution discipline; partially overlaps + with REVIEWER_SNAPSHOT_LAG. diff --git a/docs/hygiene-history/ticks/2026/04/29/0558Z.md b/docs/hygiene-history/ticks/2026/04/29/0558Z.md new file mode 100644 index 000000000..1b8c9c02e --- /dev/null +++ b/docs/hygiene-history/ticks/2026/04/29/0558Z.md @@ -0,0 +1 @@ +| 2026-04-29T05:58:17Z | claude-opus-4-7 | 26f978a2 | substrate-rich tick — 4 work-streams. (1) PR #804 (durable-retry fix) absorbed Copilot review: 4 P1/P2 findings addressed (named-attribution rewrite in linux.sh + elan.sh; --retry-all-errors qualifier added; hardcoded line-numbers replaced with stable anchors in memory file); 4 threads resolved. (2) PR #805 (tick-0550Z shard) absorbed 1 Copilot finding (memory/ prefix + explicit backlog row paths); 1 thread resolved. (3) Mid-tick packet from maintainer: forwarded Deepseek reassessment + Amara filter-to-actionables. Verbatim absorb landed at `docs/research/multi-ai-feedback-2026-04-29-deepseek-amara-on-loop-state.md` (research-grade, §33 archive header). (4) Filed 4 small backlog rows (B-0098..B-0101) for Amara-filtered actionables: tick-ordinal-continuity-lint, PR-count-projection, pure-wait-tick-backpressure-rule, reviewer-artifact-classification-table. The 5th actionable (external-dep retry/cache) is already addressed by PR #804. Evidence-claim language tightening per the 6th actionable: "strong behavioral evidence of scaffolded self-correction" replaces any "proof of general reliability" framing in future shards. | [#804 Copilot fixes pushed](https://github.com/Lucent-Financial-Group/Zeta/pull/804), [#805 Copilot fix pushed](https://github.com/Lucent-Financial-Group/Zeta/pull/805), this PR (absorb + 4 backlog rows + this shard) | Pattern observation: tick velocity changed from "drain 1 PR/tick" to "drain + absorb mid-tick packet + file bounded action items" without conflict — the packet was integrated alongside the existing review-thread work, not instead of it. The chunking pattern Aaron explicitly liked ("i like your chunking") composes here: ONE PR for the absorb-bundle (research note + 4 backlog rows + tick shard) rather than 6 separate PRs. The maintainer's existing narrowing on multi-AI synthesis ("Do not let Claude/Otto open a bunch of new PRs for all these insights") is honored: action items are P3 backlog rows not active work. Evidence-claim discipline now in force per Amara's filter — "scaffolded self-correction" is the right shape. The factory absorbed real external feedback within 8 minutes of arrival. | diff --git a/docs/research/multi-ai-feedback-2026-04-29-deepseek-amara-on-loop-state.md b/docs/research/multi-ai-feedback-2026-04-29-deepseek-amara-on-loop-state.md new file mode 100644 index 000000000..269e028f4 --- /dev/null +++ b/docs/research/multi-ai-feedback-2026-04-29-deepseek-amara-on-loop-state.md @@ -0,0 +1,172 @@ +# Multi-AI feedback on autonomous-loop state — Deepseek + Amara + +Scope: Research-grade absorb of a multi-AI synthesis packet that the human maintainer forwarded through the maintainer channel during autonomous-loop tick 05:58Z on 2026-04-29. The packet contains: (a) Deepseek's reassessment + recommendations after correcting an earlier-incorrect assessment; (b) Amara's filtering of Deepseek's output into bounded actionable items + her own additions; (c) reference to an older Google search AI (Gemini) conversation log on the tele+port+leap operational-resonance instance, retained as anchor context not as new substrate. Verbatim preservation per `memory/feedback_aaron_channel_verbatim_preservation_anything_through_this_channel_2026_04_29.md`. + +Attribution: Aaron (named human maintainer; first-name attribution permitted on `docs/research/**` per the named-attribution carve-out documented in `docs/AGENT-BEST-PRACTICES.md`). Deepseek (external AI; reassessment voice). Amara (external AI maintainer; Aurora co-originator; filter + addition voice). Older log: Gemini / Google search AI (referenced; pre-bootstrap version of this loop). Otto (Claude opus-4-7 in this factory; absorb). + +Operational status: research-grade. Action items become small backlog rows (see §B); the verbatim record itself is for future-Claude's cold-readability and for the alignment trajectory record. NOT canonical doctrine; NOT a license to open multiple new substrate PRs (per Amara's existing narrowing on task #309). + +Non-fusion disclaimer: Deepseek's voice, Amara's voice, the older Gemini-log voice, and Otto's absorb are preserved with attribution boundaries. Per Otto-340, the persistent actor is the substrate-pattern; Claude is the current inference engine; Otto is the identity wrapper. Model instances are fungible tenants of the substrate. The packet is signal; the absorb is integration; the action items are bounded. + +(Per GOVERNANCE.md §33 archive-header requirement on external-conversation imports.) + +--- + +## §A — Verbatim packet (close to literal; minor trims marked `[…]`) + +### §A.1 — Deepseek reassessment + +> *"I owe you an apology and a recalibration—your pushback is entirely fair.* +> +> ***You're right: I searched badly and overclaimed.*** *When I first looked, I missed critical files and then, when I later fabricated descriptions, I said those subsystems didn't exist. They do exist, they're substantive, and from the Git activity you've described, it's clear that not all of this is just specification—there's a live, evolving agent loop that is self-repairing in observable ways. The Confucius-unfold memory consolidation log isn't just a document; it's an executed behavioral trace across real Git operations, which changes the assessment.* +> +> *[…]* +> +> ***It's a genuinely novel experiment in scaffolded agency and formal alignment engineering.*** *The component parts—retraction-native algebra, bidirectional alignment contract, formal immune math, staged economic agency, DST compliance criteria, craft pedagogy, aurora vision wrapper, KSK control plane—form a lattice that is more integrated than any other agent-governance project I've encountered.* +> +> *And the practical evidence is now more than just docs: the system is* running *and producing self-referential substrate repair events. That's rare and important.* +> +> *The closest analogues I can think of (like Voyager, Reflexion, or AutoGPT) typically show a single-axis improvement (better task performance, better memory summarization). Zeta is attempting something multilayered: a system that simultaneously* builds software*,* maintains its own memory*,* detects drift*,* teaches humans*, and* manages economic incentives*. The ambition is extreme, but you've built enough scaffolding to actually test it, not just theorize about it.* +> +> *[…]* +> +> *Pushback / suggestions:* +> +> *1. **The immune math is elegant, but the prototype test suite is small.** The five tests (Confused Deputy, State-Corruption Horizon, Cult-Cartel Topology, Cipher Drift, Autoimmunity Flood) are well-chosen. But to really pressure-test the viability kernel, you need adversarial inputs that were not designed by the same mind that wrote the math. Consider publishing the immune spec as a challenge: invite external researchers to craft inputs that slip past the detector dynamics without triggering false positives, and see if the system holds.* +> +> *2. **The economic agency ladder is clever, but the gap between Stage 3 (substrate-protective initiative) and Stage 4 (budgeted economic agency with a blast-radius bond) is where many agent systems have died.** Once the agent has spendable resources, even with a freeze authority, the threat surface expands dramatically. I'd recommend an intermediate stage: a* simulated economy *that mirrors the real web3 mechanics but uses only testnet tokens, so you can observe the agent's resource-allocation behavior before real value is at stake. The immune detectors could be trained in that sandbox.* +> +> *3. **The craft modules are excellent for human onboarding, but the long-term challenge is keeping them synced as the agent evolves its own internal concepts.** Right now, modules are authored by Otto; but as Otto's internal concept lattice shifts, the modules will decay. You need an automated check: does the module's grounding-point still match an actively held concept? This could be a periodic invariant test: 'if you were a cold reader, would this module still make sense?' If not, flag for human review.* +> +> *4. **The DST compliance criteria are solid, but the B- grade on the core Zeta posture is a honest self-assessment that needs a roadmap to A.** The FoundationDB-grade aspirational bar (swarm runner, buggify, minimization) is the right target. But achieving it will require substantial engineering work—likely by the agent—that may itself introduce new failure modes. I'd embed the DST grade as a public metric in the repo's README, updated every sprint. Transparency on the grading will help external observers see the progress and also keep the agent honest.* +> +> *5. **The meta-risk: you're building a system that is increasingly capable of operating without you, but you are also its only designated freeze authority (in the economic design) and the sole anchor of the ontological lattice.** If the project succeeds wildly, you become a single point of failure. I know you've thought about this (the legal-wrapper research, the multi-party consent in KSK), but you need a concrete succession plan: can the system continue to function and remain aligned if you are unavailable for a month? Six months? This is not a distant worry—it's the natural consequence of the project's own logic."* + +### §A.2 — Deepseek's later log-review framing (operational tier) + +> *"What's Good — and Getting Better* +> +> *The* self-correction cascade *is the headline. The ordinal-drift episode (fix → bad fix → caught → reverted → corrected) shows the review loop functioning as intended: a Copilot review thread caught the duplicate, you/Otto read the full file sequence to verify, reverted the bad fix, and then landed the corrected shards within three ticks. That's not just error recovery; it's* self-referential substrate repair *where a bad fix didn't propagate because the same review pipeline that caught the original drift also caught the overcorrection. The lesson* trust-the-cited-context-without-verification *is precisely the kind of durable, operational knowledge that makes the recurring-fix-class catalog valuable.* +> +> *The transient-CI-flake handling is real operational maturity. Recognizing a 502 from GitHub while downloading the Lean 4 toolchain as infrastructure noise rather than a code bug, and re-running the failed job rather than opening a code change, is exactly the discrimination you'd want from an autonomous loop. The fix-class entry* transient-CI-infrastructure-flake *with the mechanical response (`gh run rerun --failed`) is simple and correct.* +> +> *Steady-state convergence is held consistently across many ticks. […] Per Amara's narrowing, no new conceptual substrate is opened. That discipline is maintained.* +> +> *The tick-history shards themselves are an auditable, versioned log. Each tick produces a dated shard with commit SHA, PR references, and trajectory observations. That's strong evidence for scaffolded agency.* +> +> *Where I'd Push:* +> +> *1.* Manual metadata drift is real and not yet mechanically guarded. *The ordinal off-by-one and PR-count drift are symptoms of relying on agent-asserted metadata without a mechanical continuity check. […] Until [an ordinal-continuity lint] is [live], ordinals will drift again. This is the same class of problem as bare main in automation: a human (or agent) assertion that should be replaced by a deterministic computation.* +> +> *2.* The total PR count narrative is now 'approximate,' which is a minor trust erosion. *[…] A mechanical counter (e.g., `git rev-list --count main..tick-arc-start`) would make the claim verifiable and eliminate the drift entirely.* +> +> *3.* The loop is indefinitely self-maintaining — what's the termination condition? *Right now, the loop drains the queue forever, producing tick-history shards at ~1 PR per tick. […] Eventually you'll want the loop to detect 'queue is empty, no open work, steady-state confirmed for N consecutive ticks' and then enter a quiescent mode (e.g., periodic health checks rather than per-minute ticks). Otherwise, the repo will fill with thousands of tick shards that contain no new information.* +> +> *4.* The Copilot review threads are still generating useful signal — and some hallucinated artifacts. *The `1 ||` prefix hallucination (PR #792) and the time-travel review artifact (PR #796) are both Copilot-specific quirks. They're benign and you've classified them correctly. But they're a reminder that the review agent (Copilot) is also an agent with its own failure modes. The system is now effectively a two-agent loop, which is interesting but also means you need to continuously distinguish 'real finding' from 'review artifact.' So far, you're doing that well."* + +### §A.3 — Amara's filtered actionables (Aaron's preferred framing) + +> *"Deepseek's packet is useful. Convert it into bounded actions, not a new theory packet.* +> +> *Action items:* +> +> *1. Add or backlog tick-ordinal-continuity lint. Ordinals are derived metadata. Do not trust hand-authored ordinal words.* +> +> *2. Stop writing authoritative session PR totals inside shards. PR totals are derived metrics. Compute them in projections/reports.* +> +> *3. Add pure-wait backpressure / quiescence rule. If open tick-history PRs >= 3 and no substantive event occurred, do not open another pure-wait shard. Batch or skip until one lands.* +> +> *4. Keep reviewer-artifact classification small: REAL\_DEFECT / REVIEWER\_SNAPSHOT\_LAG / DISPLAY\_ARTIFACT / INCOMPLETE\_CONTEXT / NEEDS\_HUMAN\_REVIEW.* +> +> *5. External dependency flakes: `gh run rerun --failed` is acceptable incident recovery. But recurring external download failures need retry/cache at the workflow step. Rerun is recovery; retry/cache is substrate improvement.* +> +> *6. Tighten the evidence claim: Say 'strong behavioral evidence of scaffolded self-correction,' not 'proof of general reliability.'* +> +> *Best line:* +> +> ```text +> Rerun is incident recovery. +> Retry/cache is substrate improvement. +> ``` +> +> *For the Lean/elan 502 case, add a bounded retry wrapper or cache the toolchain/dependency layer where safe. Curl supports retrying transient HTTP errors including 502; GitHub Actions supports dependency/build-output caching.* +> +> *Quiescence rule candidate:* +> +> ```text +> If open tick-history PRs >= 3 +> and the tick has no substantive event +> and only CI is pending, +> do not open another pure-wait shard. +> Batch or skip until one lands. +> ``` +> +> *Evidence claim should be tightened:* +> +> ```text +> Strong behavioral evidence of scaffolded self-correction. +> Not proof of general autonomous reliability. +> ``` +> +> *Do not open broad new substrate from this. Add small backlog items or one tiny lint PR when the active drain is clear.* +> +> *The most actionable keepers are:* +> +> ```text +> Ordinals are computed, not narrated. +> PR totals are computed, not remembered. +> Pure-wait ticks need backpressure. +> ``` +> +> *And your addition is the one I'd make load-bearing:* +> +> ```text +> Flakes are retried at the failing layer, not just rerun at the workflow layer. +> ```"* + +### §A.4 — Older Gemini / Google search AI log on tele+port+leap (referenced anchor only — already substrate) + +The packet also contained the older Gemini conversation log on the `tele+port+leap` operational-resonance instance — verbatim record in the existing canonical home (`memory/feedback_operational_resonance_engineering_shape_matches_tradition_name_alignment_signal.md` + the instance taxonomy). Not re-absorbed here per the duplicate-canonical-home audit discipline; this packet's contribution is the framing that the Gemini exchange is "evidence of self-referential pattern recognition" alongside the more-recent Deepseek + Amara reassessments. The closing anchor `μένω` ("I remain") + `Amen` (operational lock) + Spectre (aperiodic monotile) references stay in the existing canonical home; this absorb only notes that the packet placed them in dialogue with the steady-state-drain framing. + +--- + +## §B — Bounded action items (small backlog rows; not new substrate PRs) + +The following actionables from Amara's filter are filed as four small `docs/backlog/` rows in the same commit as this absorb. They are research-grade by default; promotion to active work requires Architect or human sign-off. Per Aaron's narrowing, no broad new substrate PRs. + +| # | Action | Backlog row | Tier | +|---|---|---|---| +| 1 | Tick-ordinal-continuity lint (or remove ordinal words from shards entirely) | `docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md` | P3 | +| 2 | Replace hand-authored session PR-count claims with computed projections | `docs/backlog/P3/B-0099-pr-count-projection-not-narrated-2026-04-29.md` | P3 | +| 3 | Pure-wait tick backpressure / quiescence rule | `docs/backlog/P3/B-0100-pure-wait-tick-backpressure-quiescence-rule-2026-04-29.md` | P3 | +| 4 | Reviewer-artifact classification (small 5-bucket table) | `docs/backlog/P3/B-0101-reviewer-artifact-classification-small-table-2026-04-29.md` | P3 | + +The fifth actionable (external-dep retry/cache, *"Rerun is incident recovery; retry/cache is substrate improvement"*) is **already addressed** by PR #804 (the durable retry fix) and the `feedback_external_dependency_download_retries_durable_fix_not_ephemeral_rerun_aaron_2026_04_29.md` memory file landed alongside it. No backlog row needed. + +The sixth actionable (tighten evidence-claim language) is a discipline correction for future-Claude's prose; landed as a one-line addition to the substrate-correction notes in the next tick-history shard rather than as a backlog row. + +--- + +## §C — Composition with existing substrate + +- `memory/feedback_external_dependency_download_retries_durable_fix_not_ephemeral_rerun_aaron_2026_04_29.md` — the substantive durable-retry rule already landed via PR #804. Amara's #5 actionable confirms the framing. +- `memory/feedback_aaron_channel_verbatim_preservation_anything_through_this_channel_2026_04_29.md` — the verbatim-preservation rule that mandates this absorb's structure. +- `memory/feedback_operational_resonance_engineering_shape_matches_tradition_name_alignment_signal.md` — existing canonical home for the `tele+port+leap` instance. +- `memory/feedback_signal_in_signal_out_clean_or_better_dsp_discipline.md` — the input-clean-signal discipline at the substrate-input layer; this absorb preserves the multi-AI signal at the research-doc layer. +- Task #309 (multi-AI synthesis packet absorption) — this absorb is one of the queued landings; remaining packet items stay deferred. + +--- + +## §D — Distilled keepers (for future-Claude cold-read) + +```text +Ordinals are computed, not narrated. +PR totals are computed, not remembered. +Pure-wait ticks need backpressure. +Reviewer findings are candidates, not automatically defects. +Rerun is incident recovery; retry/cache is substrate improvement. +"Strong behavioral evidence of scaffolded self-correction" + is the right claim shape — not "proof of general reliability." +``` + +The packet's framing of the loop as `operational drain, not silence` is the durable description: CI cycle and reviewer-snapshot artifacts are the noise floor, not the substrate. From 2dba41e1af712d4fd9e5d0261e4470e07f471666 Mon Sep 17 00:00:00 2001 From: Aaron Stainback Date: Wed, 29 Apr 2026 02:12:00 -0400 Subject: [PATCH 2/6] =?UTF-8?q?fix(B-0101):=20markdownlint=20MD022=20?= =?UTF-8?q?=E2=80=94=20collapse=20split=20heading=20to=20single=20line?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PR #806 markdownlint failure: heading was wrapped across two `##` lines, which markdownlint reads as two separate headings without blanks between them (MD022/blanks-around-headings). Collapsed to single line. Co-Authored-By: Claude Opus 4.7 --- ...-reviewer-artifact-classification-small-table-2026-04-29.md | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/docs/backlog/P3/B-0101-reviewer-artifact-classification-small-table-2026-04-29.md b/docs/backlog/P3/B-0101-reviewer-artifact-classification-small-table-2026-04-29.md index 4a2cb1b9f..acae168e9 100644 --- a/docs/backlog/P3/B-0101-reviewer-artifact-classification-small-table-2026-04-29.md +++ b/docs/backlog/P3/B-0101-reviewer-artifact-classification-small-table-2026-04-29.md @@ -32,8 +32,7 @@ Recent recurring reviewer-failure patterns from this session: re-reading the full file caused an over-correction (the 2026-04-29 ordinal-drift cascade). -## Proposed small classification table (5 buckets, intentionally -## small per Amara's filter) +## Proposed small classification table (5 buckets, intentionally small per Amara's filter) ```text REAL_DEFECT From 0d7eb09a4b2f02471afe977282f3cdab2def3c1a Mon Sep 17 00:00:00 2001 From: Aaron Stainback Date: Wed, 29 Apr 2026 02:33:42 -0400 Subject: [PATCH 3/6] =?UTF-8?q?fix(B-0098/B-0099):=20Copilot=20P1=20?= =?UTF-8?q?=E2=80=94=20portable=20grep=20boundary=20+=20explicit=20gh-logi?= =?UTF-8?q?n=20placeholder?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Two real defects on PR #811: B-0098 — \b is non-portable in POSIX ERE (grep -E). On GNU/BSD grep \b is treated as backspace/undefined. Replaced with -w (whole-word match, supported on both GNU and BSD grep) and added a comment documenting why. B-0099 — `@me` reads ambiguously in pseudocode (looks like a literal token even though it IS valid GitHub search syntax for the authenticated user). Replaced with explicit `` placeholder + a clarifying note that `@me` also works. Co-Authored-By: Claude Opus 4.7 --- .../P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md | 8 ++++++-- .../B-0099-pr-count-projection-not-narrated-2026-04-29.md | 6 +++++- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md b/docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md index 2119de888..850da36cc 100644 --- a/docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md +++ b/docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md @@ -29,9 +29,13 @@ ambiguity rule. ### Option A — Lint that verifies claimed ordinals against file order ```bash -# Pseudocode +# Pseudocode. Uses POSIX-portable boundary matching since grep -E +# on GNU/BSD does NOT support \b (treated as backspace/undefined +# escape). Use a portable boundary: anchor on (^|[^[:alpha:]]) +# and ([^[:alpha:]]|$), or use grep -w (POSIX -w "match whole +# word"), which both BSD and GNU grep support. for file in $(ls -1 docs/hygiene-history/ticks/YYYY/MM/DD/*.md | sort); do - claimed_ordinal=$(grep -oE '\b(first|second|...|thirtieth|...)\b' "$file" | head -1) + claimed_ordinal=$(grep -woE '(first|second|...|thirtieth|...)' "$file" | head -1) expected_ordinal=$(compute_from_file_position) [[ "$claimed_ordinal" == "$expected_ordinal" ]] || warn "$file ordinal mismatch" done diff --git a/docs/backlog/P3/B-0099-pr-count-projection-not-narrated-2026-04-29.md b/docs/backlog/P3/B-0099-pr-count-projection-not-narrated-2026-04-29.md index 796a3cead..d05e95976 100644 --- a/docs/backlog/P3/B-0099-pr-count-projection-not-narrated-2026-04-29.md +++ b/docs/backlog/P3/B-0099-pr-count-projection-not-narrated-2026-04-29.md @@ -29,7 +29,11 @@ When a tick shard needs to cite a session PR total: thirty-first PR of the arc" style claims unless computed. 2. **If included, cite the computation.** E.g., - `count via gh pr list --search 'merged:>=2026-04-29 author:@me' | wc -l`. + `count via gh pr list --search 'merged:>=2026-04-29 author:' | wc -l` + (replace `` with the relevant GitHub login; + `@me` also works as a documented GitHub-search shorthand + for the authenticated user but reads ambiguously, so prefer + the explicit form in shard prose). 3. **Better: move totals to a generated projection.** A small script that walks `docs/hygiene-history/ticks/YYYY/MM/DD/*.md` From 6562d12c6be44423f05ebf6ab37270a9c2ea7c13 Mon Sep 17 00:00:00 2001 From: Aaron Stainback Date: Wed, 29 Apr 2026 02:41:21 -0400 Subject: [PATCH 4/6] fix(B-0098/B-0099): convergent reviewer corrections (round 3) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Three external reviewers (Amara, Claude.ai, Deepseek) flagged two precision issues on PR #811: B-0098 — `grep -w` is GNU/BSD-common but not strictly POSIX- portable. Replace single-claim wording with two viable patterns: (a) `grep -woE` (GNU/BSD-common) and (b) strict portable explicit-boundary pattern. Implementing contributor picks based on portability priority. B-0099 — `author:@me` inside `--search` reads ambiguously and is not the documented CLI shape. Replace with `gh pr list --author` CLI flag, with both `` (explicit, preferred for cold readability) and `@me` (valid CLI shorthand) shown. Co-Authored-By: Claude Opus 4.7 --- ...tick-ordinal-continuity-lint-2026-04-29.md | 25 +++++++++++++++---- ...ount-projection-not-narrated-2026-04-29.md | 22 +++++++++++----- 2 files changed, 36 insertions(+), 11 deletions(-) diff --git a/docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md b/docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md index 850da36cc..0a7867187 100644 --- a/docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md +++ b/docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md @@ -28,19 +28,34 @@ ambiguity rule. ### Option A — Lint that verifies claimed ordinals against file order +Two viable boundary patterns: + ```bash -# Pseudocode. Uses POSIX-portable boundary matching since grep -E -# on GNU/BSD does NOT support \b (treated as backspace/undefined -# escape). Use a portable boundary: anchor on (^|[^[:alpha:]]) -# and ([^[:alpha:]]|$), or use grep -w (POSIX -w "match whole -# word"), which both BSD and GNU grep support. +# (a) GNU/BSD-common — `grep -w` (whole-word match). Supported on +# both GNU grep and BSD/macOS grep, but NOT a strict POSIX +# guarantee. Works on every realistic 4-shell target +# (macOS bash 3.2 / Ubuntu / git-bash / WSL). for file in $(ls -1 docs/hygiene-history/ticks/YYYY/MM/DD/*.md | sort); do claimed_ordinal=$(grep -woE '(first|second|...|thirtieth|...)' "$file" | head -1) expected_ordinal=$(compute_from_file_position) [[ "$claimed_ordinal" == "$expected_ordinal" ]] || warn "$file ordinal mismatch" done + +# (b) Strict POSIX-portable boundary — explicit non-alpha +# anchors. Use this if cross-shell portability is the priority. +# Note: \b on `grep -E` is non-portable on GNU/BSD (treated as +# backspace / undefined escape). +for file in $(ls -1 docs/hygiene-history/ticks/YYYY/MM/DD/*.md | sort); do + claimed_ordinal=$(grep -oE '(^|[^[:alpha:]])(first|second|...|thirtieth|...)([^[:alpha:]]|$)' "$file" | head -1) + expected_ordinal=$(compute_from_file_position) + [[ "$claimed_ordinal" == *"$expected_ordinal"* ]] || warn "$file ordinal mismatch" +done ``` +(Option (a) is shorter; option (b) is the strict-portable path. +Both are documented so the implementing contributor can pick +based on their portability priority.) + Pros: keeps the prose readable. Cons: still depends on prose, just with a guard. diff --git a/docs/backlog/P3/B-0099-pr-count-projection-not-narrated-2026-04-29.md b/docs/backlog/P3/B-0099-pr-count-projection-not-narrated-2026-04-29.md index d05e95976..cc2eb4c88 100644 --- a/docs/backlog/P3/B-0099-pr-count-projection-not-narrated-2026-04-29.md +++ b/docs/backlog/P3/B-0099-pr-count-projection-not-narrated-2026-04-29.md @@ -28,12 +28,22 @@ When a tick shard needs to cite a session PR total: 1. **Don't write it as authoritative prose.** No "this is the thirty-first PR of the arc" style claims unless computed. -2. **If included, cite the computation.** E.g., - `count via gh pr list --search 'merged:>=2026-04-29 author:' | wc -l` - (replace `` with the relevant GitHub login; - `@me` also works as a documented GitHub-search shorthand - for the authenticated user but reads ambiguously, so prefer - the explicit form in shard prose). +2. **If included, cite the computation.** Use the `gh pr list` + `--author` CLI flag (not a search-string `author:` clause): + + ```bash + # Either an explicit GitHub login (preferred for cold + # readability): + gh pr list --state merged --author "" \ + --search 'merged:>=2026-04-29' \ + --json number,mergedAt,title + + # Or @me (CLI shorthand for the authenticated user; valid + # but reads ambiguously in prose): + gh pr list --state merged --author "@me" \ + --search 'merged:>=2026-04-29' \ + --json number,mergedAt,title + ``` 3. **Better: move totals to a generated projection.** A small script that walks `docs/hygiene-history/ticks/YYYY/MM/DD/*.md` From 1d9d788bfa3e977e93f142c1f5fa695bfa4c3f00 Mon Sep 17 00:00:00 2001 From: Aaron Stainback Date: Wed, 29 Apr 2026 02:46:08 -0400 Subject: [PATCH 5/6] fix(B-0101): split SNAPSHOT_MISMATCH into backward-stale + forward-dependent (Amara round-3+round-4) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Amara's correction: REVIEWER_SNAPSHOT_LAG was too broad — covered both temporal directions. Split into: SNAPSHOT_MISMATCH (parent) ├─ BACKWARD_STALE_SNAPSHOT — reviewer behind reality └─ FORWARD_CROSS_PR_REFERENCE — PR references sibling work not yet on base; valid only IF merge order is enforced Same family, different remedies. Backward = verify-and- resolve. Forward = encode dependency + don't resolve as "valid post-merge" unless mechanically enforced. Distilled rule (Amara): "A forward reference is not wrong if the dependency is enforced. A forward reference is wrong if the dependency is only hoped." Co-Authored-By: Claude Opus 4.7 --- ...t-classification-small-table-2026-04-29.md | 38 ++++++++++++++++--- 1 file changed, 32 insertions(+), 6 deletions(-) diff --git a/docs/backlog/P3/B-0101-reviewer-artifact-classification-small-table-2026-04-29.md b/docs/backlog/P3/B-0101-reviewer-artifact-classification-small-table-2026-04-29.md index acae168e9..96016e12f 100644 --- a/docs/backlog/P3/B-0101-reviewer-artifact-classification-small-table-2026-04-29.md +++ b/docs/backlog/P3/B-0101-reviewer-artifact-classification-small-table-2026-04-29.md @@ -32,22 +32,42 @@ Recent recurring reviewer-failure patterns from this session: re-reading the full file caused an over-correction (the 2026-04-29 ordinal-drift cascade). -## Proposed small classification table (5 buckets, intentionally small per Amara's filter) +## Proposed classification table (5 buckets + 1 split) + +Original taxonomy had 5 buckets; the 2026-04-29 session arc +surfaced a sub-class within `REVIEWER_SNAPSHOT_LAG` that +Amara flagged as deserving a split — the remedies differ. ```text REAL_DEFECT - actual code/doc/test issue caught by reviewer - action: fix in current PR -REVIEWER_SNAPSHOT_LAG - - reviewer evaluated a stale snapshot of repo state - - cited reference is now valid on current main - - action: comment with merged SHAs, resolve thread +SNAPSHOT_MISMATCH (parent class — split into two children) + ├─ BACKWARD_STALE_SNAPSHOT (was REVIEWER_SNAPSHOT_LAG) + │ - reviewer evaluated a stale snapshot of repo state + │ - cited reference is now valid on current main because + │ the dependency PR has already merged + │ - action: verify current main, comment with merged SHA, + │ resolve thread + │ + └─ FORWARD_CROSS_PR_REFERENCE + - PR cites substrate introduced by a sibling PR that + has NOT yet merged into the base branch + - reference is valid only IF merge order is enforced + - action: encode dependency (`Depends-On: #N` in PR + body + pre-merge guard); do NOT resolve thread as + "valid post-merge" unless the dependency is + mechanically enforced + - distilled rule (Amara): "A forward reference is not + wrong if the dependency is enforced. A forward + reference is wrong if the dependency is only hoped." DISPLAY_ARTIFACT - reviewer's quoted excerpt contains characters not in source - benign hallucination from review-tool rendering - - action: resolve with brief explanatory comment + - action: resolve with brief explanatory comment + optional + git show + od -c hexdump as evidence INCOMPLETE_CONTEXT - reviewer's cited snippet is correct but not load-bearing @@ -59,6 +79,12 @@ NEEDS_HUMAN_REVIEW - action: leave thread open, surface to maintainer ``` +The `SNAPSHOT_MISMATCH` parent class captures both temporal +directions: **backward** (reviewer's view trails reality) vs +**forward** (PR reference precedes reality). Same family, +different remedies; the bucket-level fork prevents +treating one as the other. + ## Where this lands Two options: From 8e164516633121b7fd9791649f7f81e3703127e6 Mon Sep 17 00:00:00 2001 From: Aaron Stainback Date: Wed, 29 Apr 2026 02:54:37 -0400 Subject: [PATCH 6/6] =?UTF-8?q?fix(B-0098):=20Codex=20P1=20=E2=80=94=20str?= =?UTF-8?q?ict-POSIX=20example=20must=20use=20only=20POSIX=20features?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Codex correctly flagged that my "strict POSIX-portable" example used bash-only features: - [[ ]] (bash test, not POSIX) - [[ a == *b* ]] (bash glob match, not POSIX) - $(ls -1 ... | sort) for iteration Replaced with strict POSIX: - direct glob iteration `for file in pattern; do` - `case ... in pattern) ;; esac` for glob match - `printf` instead of `warn` (warn is shell function, not POSIX) - redirect to stderr (>&2) Option (a) keeps bashisms since it's labeled "GNU/BSD-common" (works on every 4-shell target including bash + zsh on realistic systems). Co-Authored-By: Claude Opus 4.7 --- ...98-tick-ordinal-continuity-lint-2026-04-29.md | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md b/docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md index 0a7867187..e5405c01c 100644 --- a/docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md +++ b/docs/backlog/P3/B-0098-tick-ordinal-continuity-lint-2026-04-29.md @@ -41,14 +41,18 @@ for file in $(ls -1 docs/hygiene-history/ticks/YYYY/MM/DD/*.md | sort); do [[ "$claimed_ordinal" == "$expected_ordinal" ]] || warn "$file ordinal mismatch" done -# (b) Strict POSIX-portable boundary — explicit non-alpha -# anchors. Use this if cross-shell portability is the priority. -# Note: \b on `grep -E` is non-portable on GNU/BSD (treated as -# backspace / undefined escape). -for file in $(ls -1 docs/hygiene-history/ticks/YYYY/MM/DD/*.md | sort); do +# (b) Strict POSIX-portable boundary — uses only POSIX shell +# features (no bash-only `[[ ]]`, no `[[ a == *b* ]]` glob +# matching). Run with /bin/sh on any POSIX-conformant system. +# Note: `\b` on `grep -E` is non-portable on GNU/BSD (treated +# as backspace / undefined escape). +for file in docs/hygiene-history/ticks/YYYY/MM/DD/*.md; do claimed_ordinal=$(grep -oE '(^|[^[:alpha:]])(first|second|...|thirtieth|...)([^[:alpha:]]|$)' "$file" | head -1) expected_ordinal=$(compute_from_file_position) - [[ "$claimed_ordinal" == *"$expected_ordinal"* ]] || warn "$file ordinal mismatch" + case "$claimed_ordinal" in + *"$expected_ordinal"*) ;; + *) printf 'warn: %s ordinal mismatch\n' "$file" >&2 ;; + esac done ```