Skip to content

ci: upgrade Archfit Action to v3.1.1 - #401

Merged
alexei-led merged 3 commits into
masterfrom
ci/upgrade-archfit-action
Nov 3, 2026
Merged

alexei-led merged 3 commits into
masterfrom
ci/upgrade-archfit-action

Conversation

@alexei-led

@alexei-led alexei-led commented Nov 3, 2026 •

Copy link
Copy Markdown
Owner

Approval and merge — two separate steps

1. Approve this workflow update in Archfit: https://app.coupling.dev/r/alexei-led/pumba/pull/401

2. After the refreshed checks pass, merge on GitHub: #401

Approval authorizes the exact proposed workflow version. It does not merge or activate the update. If this exact commit is already approved, do not approve it again; wait for the refreshed result.


Change

  • Pin Archfit Action v3.1.1 by immutable commit SHA.
  • Add merge_group checks_requested while preserving existing triggers and permissions.
  • Keep the current architecture policy, accepted debt, labels and App gate mode unchanged.

Owner steps

  1. Open the Archfit App from the archfit/state check Details link.
  2. Choose Approve for this exact commit. Approval does not merge the PR.
  3. Wait for the refreshed Archfit check and CI, then merge this PR on GitHub.

Verification

  • actionlint and zizmor passed for the workflow.
  • archfit check --format agent: change.status=pass, next_action=none (exit 2; existing repository attention remains).
  • git diff --check passed. No application code changed.
  • No live merge-queue smoke test was run.

@greptile-apps

greptile-apps Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] No established issue blocks merging this workflow update.

Summary

Updates the pinned Archfit Action to v3.1.1.

  • Archfit reports now cover commits in the merge queue.
  • The workflow runs Archfit Action v3.1.1.

No actionable issues were established. The previous and current commits have different merge bases, so the review covered the full PR changes.

Reviews (3) · Last reviewed commit: "Merge branch 'master' into ci/upgrade-ar..." · Reviewed by Greptile

@archfit-review

archfit-review Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Archfit: Architecture gate passed · attention needed

No required architecture change requested in the evaluated scope. Checked with engine v3.4.0 and policy digest 31f9f70260ad.

PR impact: 0 introduced findings, 0 pre-existing, 0 unknown. Finding origin does not establish that each module pair or listed source site was added.

Next: Advisories never block. Review the listed dimensions and diagnostics on the archfit/state check.

Details in Archfit

Architecture approved

Approved by alexei-led. Evaluation and merge are separate.

Why: this pull request changes protected architecture inputs:

  • .github/workflows/archfit.yaml — workflow

Candidate head: ad0f53ff6af27ed18d5f6adaa1b1a5bda97ca54a. A new push needs a new decision.

Who can decide: any one current eligible architecture owner or owner-team member named on the protected settings ref; not everyone listed. Agents never decide.

Configured owners:

  • alexei-led

Protected settings source: .archfit-app.yaml at commit e1d9d6da148fe30f52e3391a43a67c52dce45947

Gate mode: required. Archfit requires an owner approval before its gate can pass. GitHub's branch rules decide whether this blocks merging.

GitHub does not accept a pull request author's own review. An eligible sole owner who opened the PR can decide in Archfit App.

Review changes and decide in this exact Archfit PR page

This decision does not merge or publish the pull request. A new push needs a new candidate decision.
The App PR page shows the current check; a lower result in this comment may describe an earlier commit.

HEAD ad0f53ff6af27ed18d5f6adaa1b1a5bda97ca54a · Run 41697547903 · attempt 2 · archfit/state check 126136479210 The App link shows the current PR result/recovery diagnostics, not a historical snapshot.

Evidence and inputs

Evidence gaps

Declared rule checks: 6/6 evaluated. Implicit accepted-reference metric checks: inventory not reported. Accepted reference: comparable.

Reports are customer-attested. Upload authentication does not independently verify report content.

For coding agents

Result identity, as the engine states it: engine v3.4.0, policy digest 31f9f70260ad6c11e841f51ed22f3fb02527003989409392e9165ab263fa0b57, baseline digest 37a80f2fca20dafeb1f601993d97015c6abe4e665833099ec48a91df7b451134, head tree 047068fc4535a94b97bbe7b9621c0f3da9bdeba2, base e1d9d6da148fe30f52e3391a43a67c52dce45947.

Agent tasks

No automated source-repair task supplied.

[]

@github-actions

github-actions Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Test Results

2 395 tests   2 394 ✅  12s ⏱️
   22 suites      1 💤
    1 files        0 ❌

Results for commit 1e06fbd.

♻️ This comment has been updated with latest results.

@codecov

codecov Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.70%. Comparing base (e02f416) to head (ad0f53f).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #401      +/-   ##
==========================================
+ Coverage   94.67%   94.70%   +0.02%     
==========================================
  Files          98       98              
  Lines        3359     3359              
==========================================
+ Hits         3180     3181       +1     
+ Misses        179      178       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@archfit-review

archfit-review Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Archfit architecture decision: Approve architecture change

  • By: @alexei-led
  • Candidate commit: 92e61d018306a1e5fe976e272e75442ded86a57f
  • Protected settings commit: e1d9d6da148fe30f52e3391a43a67c52dce45947
  • Recorded: 2026-11-03T18:47:08Z

This decision does not merge or publish the pull request. A new push needs a new candidate decision.

@alexei-led
alexei-led enabled auto-merge November 3, 2026 18:50
@archfit-review

archfit-review Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Archfit architecture decision: Approve architecture change

  • By: @alexei-led
  • Candidate commit: ad0f53ff6af27ed18d5f6adaa1b1a5bda97ca54a
  • Protected settings commit: e1d9d6da148fe30f52e3391a43a67c52dce45947
  • Recorded: 2026-11-03T18:55:27Z

This decision does not merge or publish the pull request. A new push needs a new candidate decision.

@alexei-led
alexei-led merged commit 17559f0 into master Nov 3, 2026
17 of 18 checks passed
@alexei-led
alexei-led deleted the ci/upgrade-archfit-action branch November 3, 2026 18:59
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