ci: restore supported standard GitHub-hosted runners - #17
Conversation
The personal-account-owned repository cannot admit the organization-only 32/96-core larger-runner labels introduced by 10f99bc. Reverse that topology and its 0012dd1 worker tuning, restoring the prior split Python and JS coverage, native OS lanes, standard Nix/Rust/E2E/Docker runners, and arm64 hosted runner without changing permit or canary source.
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. |
૮ >ﻌ< ა ci reviewran on 5d5d4ba — ci: restore supported standard GitHub-hosted runners ❌ Job failuresDocs Site / docs-site-checks · View jobJob Docs Site / docs-site-checks failed. JS & TS checks / apps/desktop / check:test:ui:shard-2of3 · View jobJob JS & TS checks / apps/desktop / check:test:ui:shard-2of3 failed. JS & TS checks / tests-js / check · View jobJob JS & TS checks / tests-js / check failed. OS-specific tests / Windows-only tests · View jobJob OS-specific tests / Windows-only tests failed. Python lints / Windows footguns (blocking) · View jobJob Python lints / Windows footguns (blocking) failed. Python tests / Run tests slice 2/12 · View jobJob Python tests / Run tests slice 2/12 failed. Python tests / Run tests slice 5/12 · View jobJob Python tests / Run tests slice 5/12 failed. Python tests / Run tests slice 8/12 · View jobJob Python tests / Run tests slice 8/12 failed. Python tests / Run tests slice 9/12 · View jobJob Python tests / Run tests slice 9/12 failed.
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5d5d4bae32
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "check:test:desktop:all": "npm run test:desktop:all", | ||
| "check:lint": "npm run typecheck && npm run lint", | ||
| "check": "npm run check:lint && npm run test:ui && npm run test:desktop:platforms && npm run test:desktop:all && npm run check:test:plugins", | ||
| "check": "npm run check:lint && npm run test:ui && npm run test:desktop:platforms && npm run test:desktop:all", |
There was a problem hiding this comment.
When a contributor or automation runs the aggregate npm run check --workspace=apps/desktop, the plugin suite no longer executes even though check:test:plugins remains defined. The new CI matrix invokes that sub-check separately, but other consumers of the package's canonical aggregate command now report success without running src/plugins/*/tests/*.test.mjs; retain npm run check:test:plugins in this command.
Useful? React with 👍 / 👎.
RecursiveIntell
added
the
ci-reviewed
1 participant
Summary
Restore Ares CI to its previously admitted standard GitHub-hosted runner topology.
The repository is owned by the personal GitHub account
RecursiveIntell, not an organization. The configured larger-runner labels do not admit under that ownership, leaving affected CI/Nix jobs queued without a runner assignment.This change:
ubuntu-24.04-armfor the arm64 Docker lane;Trigger evidence
mainand unrelated PRs.runner_id: 0with no runner name.ubuntu-latestandmacos-latestjobs in the same runs completed.User; larger hosted runners require qualifying organization or enterprise ownership.Local validation
run_tests.sh --filesexecution passed 8 tests across 2 files;git diff --checkpasses.Two earlier local test-command attempts stopped before execution because the isolated worktree/runtime venv lacked pytest; the successful rerun used the existing Ares development venv. Hosted CI remains the decisive execution gate.
Publication boundary
Public Evidence Editor outcome for commit/push/open PR:
APPROVE_WITH_QUALIFIERS.Receipt:
/home/sikmindz/.ares/profile-collaboration/receipts/20260831T055439Z-fad4b9e0Merge is intentionally deferred until standard hosted runners admit the jobs, affected checks reach acceptable terminal states, matrix cardinality is confirmed, and a renewed public closeout review passes.
Non-claims
This PR does not claim that CI is already fixed, all tests passed, performance is equivalent, the baseline Windows-footgun debt is resolved, or the paired permit contract is validated, merged, or live.