Skip to content

Review #37 coverage diagnostics workflow delta under manual authority #187

Description

@seonghobae

Exact owning boundary

Canonical HTTP reconstruction PR #37 remains the owner of the HTTP/1.1 source/tests/docs at exact contributor head c8e0f1de85caf31299eb49730510aabf5c8f379a, and it still contains the pre-existing .github/workflows/ci.yml coverage-diagnostics delta.

Protected main has since advanced to b05d5acca82b9d916ada2c8e82f59f92a89817e1. A fresh PR read now reports #37 open, Ready, but non-mergeable, with its recorded base snapshot still at predecessor main 0841d2ab3d8b5e60a03c0a8e818cf438e2716829. Therefore the old native CI/security/coverage results remain exact evidence for c8e0f1de85caf31299eb49730510aabf5c8f379a against that predecessor base only; they are not current-live-base merge evidence.

Fresh source inspection still identifies the workflow delta as the governance-owned boundary: the Record uncovered production lines step uses Bash pipefail plus tee so cargo llvm-cov report failure propagation is preserved while diagnostics are materialized.

Why this needs a separate owner decision

Protected-main AGENTS.md explicitly says scheduled agents may not alter workflows. The scheduled OriginWeave writer therefore cannot safely adopt, revert, rewrite, or otherwise mutate this workflow delta while repairing #37. Product-code churn or synthetic restacking that hides the workflow authority problem would be incorrect.

An authorized manual/non-scheduled owner must inspect the then-current #37 workflow diff and decide whether the coverage-diagnostic hardening should remain in #37, move through an independently authorized workflow change, or be reverted without weakening failure propagation. Because protected main moved, that decision must also reconstruct/revalidate the branch against the live base without treating predecessor checks as transferable.

Historical exact-head evidence

On exact head c8e0f1de85caf31299eb49730510aabf5c8f379a against predecessor main 0841d2ab3d8b5e60a03c0a8e818cf438e2716829:

  • CI run 32652915332: success;
  • Rust contracts job 97227194086: success;
  • Production coverage job 97227194202: success with 492/492 functions, 5956/5956 regions, 4398/4398 lines, 612/612 branches = 100%;
  • exact coverage artifact 9496705860, digest sha256:02c7749a939e21315b2afeb2bff94740b27a1a75bc2a5a7ececcfe8e2e2bb7ff;
  • SAST Semgrep run 32652915292: success; and
  • Security Scan run 32652915253: success.

Those results remain useful branch-history evidence only. They are not promoted as live-base or merge evidence after main advanced.

The two unresolved governance threads remain:

  • PRRT_kwDOTulPlM6a8XuD; and
  • PRRT_kwDOTulPlM6a8eIj.

Both point to .github/workflows/ci.yml and remain unresolved because this scheduled writer is not authorized to mutate that workflow or resolve a finding it cannot address.

Acceptance evidence

Before any integration decision for #37:

  • refetch exact protected main and exact feat(http): rebuild bounded HTTP/1.1 authority on current main #37 head;
  • inspect the intervening protected-main delta and reconstruct/revalidate feat(http): rebuild bounded HTTP/1.1 authority on current main #37 against that live base without force-push or predecessor-evidence transfer;
  • verify the workflow delta and current repository authority;
  • ensure cargo llvm-cov report failure still propagates;
  • preserve exact production function/line/region/branch coverage diagnostics and enforcement;
  • remove or independently authorize the workflow delta through an eligible governance path;
  • resolve the two workflow threads only after the authority defect is actually addressed; and
  • regenerate applicable exact-head CI/security/scanner/review evidence on the resulting current branch/base relationship.

This issue is owner routing only. It does not authorize this scheduled writer to modify .github/workflows/ci.yml, merge #37, or treat stale/predecessor evidence as passing current proof.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions