-
Notifications
You must be signed in to change notification settings - Fork 0
fix(reliability): preflight detached autosave digest length before regex scanning #214
Copy link
Copy link
Open
Labels
area: apiAPI, protocol, event, or external contractAPI, protocol, event, or external contractarea: authAuthentication, authorization, identity, or tenant isolationAuthentication, authorization, identity, or tenant isolationarea: ci-cdCI, GitHub Actions, checks, release, or supply chainCI, GitHub Actions, checks, release, or supply chainarea: dataDatabase, schema, migration, ETL, or lineageDatabase, schema, migration, ETL, or lineagebugSomething isn't workingSomething isn't workingpriority: mediumNormal-priority or P2 workNormal-priority or P2 workstatus: blockedBlocked by conflict, dependency, or required prerequisiteBlocked by conflict, dependency, or required prerequisitetype: bugDefect or incorrect behaviorDefect or incorrect behavior
Description
Activity
Metadata
Metadata
Assignees
Labels
area: apiAPI, protocol, event, or external contractAPI, protocol, event, or external contractarea: authAuthentication, authorization, identity, or tenant isolationAuthentication, authorization, identity, or tenant isolationarea: ci-cdCI, GitHub Actions, checks, release, or supply chainCI, GitHub Actions, checks, release, or supply chainarea: dataDatabase, schema, migration, ETL, or lineageDatabase, schema, migration, ETL, or lineagebugSomething isn't workingSomething isn't workingpriority: mediumNormal-priority or P2 workNormal-priority or P2 workstatus: blockedBlocked by conflict, dependency, or required prerequisiteBlocked by conflict, dependency, or required prerequisitetype: bugDefect or incorrect behaviorDefect or incorrect behavior
Current authoritative state
The original statement that detached digest preflight remained unbounded on Draft #184 is stale. Draft #184 remains the canonical single writer for
src/autosave/evidenceValidation.ts; protected shipped truth ismain@3b38ead2d00f44eb578d0689087b9293b3dabe1eand current exact #184 head is62f406c2321492ce3161d2f72f45794bdfde9034.#184 now explicitly
Closes #214. Its test-first lineage records REDd7d8d879941fb08ec0597f394cc4d3a6e629a7de, where an otherwise-shaped 65-code-unit digest reached regex scanning, followed by GREEN6d75be1826442531952c51698bf19e5ab54cfa53, which requiresdigestHex.length === 64before the lowercase SHA-256 regex. Current production retains that preflight together with the #183 array/traversal resource repairs, descriptor-safe frozen-record validation and exact strong-ETag equality.Current repair contract
digestHexbeforeLOWERCASE_SHA256_DIGEST.test(...)."sha256-<digest>"strong entity-tag matching.Evidence / integration boundary
Current exact #184 head is Draft and mechanically mergeable, but protected integration remains blocked by #118's
v0.6.0boundary plus then-live central workflows and qualifying independent approval. The PR body records current-head repository verification lineage but live workflow/review state must be freshly refetched before any lifecycle action; pending/queued/skipped/cancelled/absent/stale/predecessor/status-only/model-only evidence is non-passing.Keep this issue open until #184 integrates under live governance. Do not create a competing
evidenceValidation.tswriter, transfer predecessor evidence, self-approve, weaken gates, move protected main or claim active-PR behavior as shipped.