[danieljoppi]

Summary

Adds the check runs core-test and core-format (from .github/workflows/core.yml, added by #25) to required_checks in .github/org-sync.toml [settings]. After this, a PR cannot merge while core/ fails its strict-warning build, its tests (plain and under ASan + UBSan) or clang-format. The names are the exact check-run names on #25's head commit.

Merge order: merge this only after #25. This PR is based on main, not on #25's branch, so the diff stays at one line. Until core.yml is on main, those checks never report, and requiring them would block every PR.

Checks

• tomllib parses the file: required_checks = ['check / org-sync', 'headers / licence', 'title / title', 'core-test', 'core-format'].
• The check-run names were confirmed with gh api repos/swarm-routing/HOPNET/commits/<#25 head>/check-runs: core-test and core-format, both success. feat(core): the intra-zone half — internal ants, intrart, eqs. 4.1-4.3, neighbour discovery #25 keeps both job names unchanged after it drops the vendored swarm-core copy.

---

Generated by Claude Code

Summary by CodeRabbit

• Chores
  • Added core-test and core-format to the required checks.

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

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

• Source mode: Auto
• Repository: swarm-routing/HOPNET
• Branch: main
• File: .coderabbit.yaml
• Commit: 049ca56
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 93885546-5e26-46e3-a314-0a3e5a60cab3

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

No actionable comments were generated in the recent review.

Walkthrough

The org-sync configuration now requires the core-test and core-format checks. The existing required checks remain unchanged.

Changes: Required checks

Layer / File(s)	Summary
Add core checks .github/org-sync.toml	The required checks list now includes core-test and core-format. The existing required checks remain in the list.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Possibly related issues

• [epic][protocol] HOPNET as a standalone hexagonal protocol (later: not scheduled) #3: The issue requires HOPNET to be green in CI. This change makes the core test and formatting checks required.
• [protocol] HOPNET core/ + ns-3 module hopnet, anchored against AntHocNet #5: The issue requires the core/ implementation to be green in CI. This change requires the core-test and core-format checks.

Possibly related PRs

• swarm-routing/HOPNET#24: That PR adds the required checks list that this change updates.
• swarm-routing/HOPNET#25: That PR adds the workflow jobs whose names this change makes required.

Merge Risk: Minimal · previous review af2a2

Both core checks become required. Merge after #25 supplies them; its workflow conditions remain unverified.

Security architecture risk: Low · up to af2a2

The change strengthens merge requirements, but safe activation depends on deploying both check producers first. Their coverage and the recovery path if they do not report remain unverified.

Retained concerns

• Low · reliability · inferred: If the expanded policy is activated before both check producers are available for governed pull requests, the merge gate can become unsatisfiable, potentially blocking corrective changes as well. The author documents the prerequisite merge order, but producer coverage and an effective recovery path are not established by the available evidence.

Security review details

Security Blast Radius

• inferred — The visible change affects repository merge eligibility rather than application runtime access. The exact governed branches, check-producer identity restrictions, and bypass roles depend on external policy configuration that is not available.

Trust Boundaries and Controls

• inferred — The PR strengthens the declared gate while retaining the existing external policy-management boundary. The available comparison does not establish a newly introduced bypass or authority expansion; it also does not verify which check-producing identities the applied policy trusts.

Pre-merge checks: 5

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly summarizes the main change: requiring the core checks.
Docstring Coverage		No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches

Generate unit tests (beta)

• Commit to this branch
• Create a new PR

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

Recent review info

Run configuration

• Source mode: Auto
• Repository: swarm-routing/HOPNET
• Branch: main
• File: .coderabbit.yaml
• Commit: eb98b69
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: e9f19a38-3593-4b18-9724-5c991b26616e

Commits

Reviewing files that changed from the base of the PR and between eb98b69 and af2a2d8.

Files selected for processing: 1

• .github/org-sync.toml

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 2 remain after this review.