Repository navigation
fix(ci): remove duplicate Julia timeout after PR 243 merge - #247
hyperpolymath wants to merge 1 commit into
Conversation
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 19 minutes. View limit details
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. Comment |
|
Fork governance
🟣 critical · commit
Hypatia scan: 128 finding(s)
panic-attack assail: 17 finding(s)
pons asinorum: 1 finding(s)
rhodibot (RSR compliance): not_run — enrolled, but no report-back was received
panicbot (panic recovery, error handling): not_run — enrolled, but no report-back was received
echidnabot (formal verification: proofs/): not_run — enrolled, but no report-back was received
seambot (integration health: API routes): not_run — enrolled, but no report-back was received
finishingbot (release readiness): not_run — enrolled, but no report-back was received
accessibilitybot (WCAG on the React UI): not_run — enrolled, but no report-back was received
glambot (presentation quality): not_run — enrolled, but no report-back was received
cipherbot: not_applicable — no cryptography is implemented in this repository
sustainabot: not_applicable — fleet adapter slot reserved; no adapter exists yet (gitbot-fleet README)
Scope: static findings reported by each row's tool at the named commit. Purple, red, yellow and blue grade the worst finding present; orange is evidence missing, malformed or a scanner that did not run; grey is an explicit scope exclusion with its reason. No row is a security certification. report.json, badge.json and the raw findings are in the verdict-grid artifact on the linked run. 🟢 conformant · commit
K9 live corpus (L0–L2): 1 finding(s) / scope notes
Scope: K9 components L0–L2 against contract v1.0.0 at the named standards commit. L3 signature verification and routed coordination-dialect files are not certified. Green is limited to this scope. Orange is missing evidence; yellow is grandfathered debt or warnings; grey is an explicit scope exclusion. report.json, badge.json and the validator logs are in the k9-grid artifact on the linked run. |
|
Independent validation of the exact |
|
The repair itself is validated, but this commit does not contain the repository-required |
## Fix Remove the second `timeout-minutes` key from the Julia job in `.github/workflows/ci.yml`. Retain the explicit 90-minute budget and all existing tests, relevance gates, and required context names. #243 merged as `a9fb609` with overlapping 90- and 60-minute YAML keys. Main workflow run `39990718300` consequently failed before jobs could start. This is a focused three-line repair, not a waiver of CI. This carries the exact tree of #247 (`46665dd`) onto the current session branch with the repository-required DCO signoff; #247's commit has no `Signed-off-by:`. The commit parent is current main `a9fb609`, with no merge commit and no other file changes. Duplicate replacement #252 was closed after #243 merged. ## Verification on the exact tree - Governance contract: **39/39 passed**; all **8 weakening mutations rejected** - CI producer/logger exit combinations: **4/4 passed**; K9 optional-log cases: **3/3 passed** - Scanner boundaries: working-tree and deleted-history shell credentials blocked without disclosure; Rust findings redacted; all six clean/error/finding AsciiDoc cases passed - Unique-key YAML parsing passed for **11 files**; `bash -n` passed for **84 embedded run blocks** - Equivalent local clone: full-history custom shell scan and real Gitleaks 8.30.1 working-tree/history scans clean - `git diff --check` passed - Post-#243 default-branch Scorecard run `39990720761` succeeded through analysis, SARIF upload, and artifact upload Julia/R execution requires hosted CI; no local runtime pass is claimed. #247's hosted run `39990855274` is testing the same tree, but this commit will also retain its own fresh checks. ## Scope - [x] Current-main ordinary commit with `Signed-off-by:` (DCO) - [x] Only `.github/workflows/ci.yml` changed - [x] No application, dependency, licence, generated, or vendored-file changes Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
|
Superseded by merged #254 ( |



Fix
Remove the second
timeout-minuteskey from the Julia job in.github/workflows/ci.yml. Retain the explicit 90-minute budget and all existing tests, relevance gates and required context names.Why a follow-up
#243 was rebased with user approval. Main and the PR branch then advanced concurrently, and #243 was merged as
a9fb609while the correction was being pushed. The push correctly refused to overwrite the remote update. This branch now starts from that exact merged main and contains only this three-line repair; no merged security/frontend/leaf-PD work is reverted or duplicated.The overlapping settings (
90and60) are duplicate YAML map keys, not two distinct timeouts. Main's workflow failed before jobs could start: https://github.com/hyperpolymath/MetaManifold-WebUI/actions/runs/39990718300 . This restores a valid workflow rather than waiving CI.Verification
bash -non all 84 embedded Bash steps: pass.git diff --check: pass.Local Node: 22.22.3. Julia/R/Agda are not installed locally, so hosted CI must validate the final head. Passing frontend/scanner tests from the preceding security audit are not substitutes for those runtime checks. No claim is made that main or this repair has passed integration yet.