Skip to content

fix(ci): restore Hypatia baseline behavior after #1281 - #1292

Merged
hyperpolymath merged 3 commits into
coderabbitai/autofix/ce768a6from
arena/5d9f0810-standards
Oct 19, 2026
Merged

hyperpolymath merged 3 commits into
coderabbitai/autofix/ce768a6from
arena/5d9f0810-standards

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Stacks on #1281 to address its outstanding Hypatia/baseline regressions and associated CI failures.

  • Restore acceptance of scanner baseline severities (warn, informational), warn/medium compatibility, list-valued rule_module membership, and accurate blocking counts.
  • Preserve the empty-findings [] positive control in the scanner security self-test.
  • Align the baseline validator with the authoritative schema for module arrays, optional notes, and the separate severity-override enum.
  • Add regression coverage for compatibility, blocking ranks, overrides, malformed entries, and threshold diagnostics.

K9-SVC scope is unchanged; the latest PR #1281 K9 contractile check is green. Local validation: baseline regression suite (52 checks), Hypatia blocking-gate suite, shipped-baseline filter, JSON parse, and git diff --check pass.

This is deliberately based on #1281's head branch so the fixes land in the stack before #1281 is merged.

Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-19T10:51:20.582408Z eb37721 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@coderabbitai

coderabbitai Bot commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: e9c6b092-b8fb-40b8-b677-45a66195ea03

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

…ndards

# Conflicts:
#	scripts/apply-baseline.sh

Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
@github-actions

github-actions Bot commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

DEED–K9 green

🟢 conformant · commit eb37721a2d69 · run 39240066247

Category Status Pass Fail Indeterminate N/A Since previous revision
DEED syntax 🟢 conformant 4 0 0 0 ↔ unchanged
K9 contract typecheck 🟢 conformant 1 0 0 0 ↔ unchanged
K9 validator self-test 🟢 conformant 34 0 0 0 ↔ unchanged
K9 positive/negative controls 🟢 conformant 27 0 0 0 ↔ unchanged
K9 live corpus (L0–L2) 🟢 conformant 12 0 0 2 ↔ unchanged
K9 live corpus (L0–L2): 2 finding(s) / scope notes
  • 2-protocols/k9-coordination/examples/minimal.k9 — not_applicable: Routed to another dialect; not validated as a K9 contract.
  • coordination.k9 — not_applicable: Routed to another dialect; not validated as a K9 contract.

Scope: DEED syntax and K9 contracts L0–L2. L3 signature verification and routed coordination dialects are not certified. Green is limited to this scope. Orange is missing evidence; grey is an explicit scope exclusion. Logs and report.json are in the dated conformance-evidence artifact on the linked run.

Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
@sonarqubecloud

Copy link
Copy Markdown

@hyperpolymath
hyperpolymath merged commit efdb21b into coderabbitai/autofix/ce768a6 Oct 19, 2026
23 checks passed
@hyperpolymath
hyperpolymath deleted the arena/5d9f0810-standards branch October 19, 2026 10:51
hyperpolymath added a commit that referenced this pull request Oct 19, 2026
Follow-up stacked on #1281 / #1292. The post-fix CI rerun exposed one
unrelated but blocking Gate D drift: the existing top-level
`.devcontainer` (added by #1282) was absent from `standards-map.toml`.

Adds the root machinery entry and increments `entry_count` from 128 to
129. This is an honest map update, not an exemption. `bash
scripts/check-standards-map.sh --repo .` now passes all five assertions
(129 mapped paths; 77 top-level entries mapped; count 129).

This resolves the only failed check identified on the latest #1281 run;
the Hypatia baseline validation, Hypatia scan, Repo self-tests, and
K9-SVC validation are green.

---------

Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
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