Skip to content

DO NOT MERGE — proof runs for paiml/.github Arbiter R2a artifact rows (paiml/infra#532) - #612

Closed
noahgift wants to merge 8 commits into
mainfrom
r2a-proof-do-not-merge
Closed

noahgift wants to merge 8 commits into
mainfrom
r2a-proof-do-not-merge

Conversation

@noahgift

Copy link
Copy Markdown
Contributor

Throwaway draft. It points forjar's ci job at paiml/.github@arbiter-r2a-artifact-rows to prove, on real runs, what that workflow change claims (paiml/infra#532, epic paiml/infra#932):

  1. stock forjar: junit + sccache-metrics artifacts appear, and pass/fail is unchanged
  2. forjar with its own [profile.default.junit]: the repo's file name wins
  3. a planted failing test: red with the flag, as it is without it

It gets closed, never merged, when the proofs are recorded.

🤖 Generated with Claude Code

noahgift added a commit to paiml/.github that referenced this pull request Sep 23, 2026
…-config, marker, any xml, nextest version)

The first real proof run (paiml/forjar#612, run 35886458605) staged 0
junit files. nextest ran the default profile with no fallback, and the
warning fired as designed, but it gave no reason. This makes the
warning diagnosable. Additive, only on the path that already warns.

Refs paiml/infra#532

Pmat-Ticket: PMAT-532
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
noahgift added a commit to paiml/.github that referenced this pull request Sep 23, 2026
…tored, subshell with a named warning, continue-on-error on every R2a step

Proof run 2 (paiml/forjar#612, run 35905480624) went RED because of
this workflow change. A scripted edit dropped the backslash from
`find … -exec ls {} \;`, find errored ("missing argument to -exec"), and
`sh -e` aborted the staging step. That broke the condition this change
exists under: nothing here may turn a job red.

- the escape is restored (by exact-string edit; the scripted one had
  eaten it twice);
- the diagnostic runs in a subshell whose failure is a NAMED
  ::warning::, never an abort;
- every "Arbiter R2a" step is `continue-on-error: true`: the
  GitHub-native guarantee that however an R2a step breaks, the job's
  verdict is the tests', not ours.

Verified by extracting the staging step and running it exactly as CI
does (`sh -e`): no nextest dir → rc 0 with one warning; a fresh xml →
rc 0 and 1 file staged. actionlint findings identical to main.

Refs paiml/infra#532

Pmat-Ticket: PMAT-532
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
noahgift added a commit to paiml/.github that referenced this pull request Sep 24, 2026
…s junit, and print arch, host and cargo's target_directory

Proof runs 1-3 on paiml/forjar#612 ran their test jobs on gx10-pool1,
yoga-build2 and gx10-pool3, never on intel. The same flag writes junit
locally on nextest 0.9.132 and 0.9.143, including with forjar's own
nextest.toml. The next run has to say where the file went, or that it
was never written. Additive, in the subshell that can never abort the
step.

Refs paiml/infra#532

Pmat-Ticket: PMAT-532
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@noahgift

Copy link
Copy Markdown
Contributor Author

Closed by the PR-cap triage (paiml/infra#1049). The operator capped open PRs per repo, and this repo is over its cap with no live owning session. The branch is kept, so reopen or fold this into a like PR when work resumes. List and decisions: cop-inbox/evidence/prcap-orphan-triage.md

@noahgift noahgift closed this Sep 24, 2026
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