Skip to content

GH#34286: Explain tag-push release evidence failures - #34290

Merged
alex-solovyev merged 6 commits into
mainfrom
feature/auto-20261010-224140-gh34286
Oct 10, 2026
Merged

alex-solovyev merged 6 commits into
mainfrom
feature/auto-20261010-224140-gh34286

Conversation

@alex-solovyev

@alex-solovyev alex-solovyev commented Oct 10, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Resolves #34286. Keeps the release-event default and publication guards unchanged. When the named workflow has no matching successful release run, reports successful events for the exact merge SHA and suggests --event push only for a successful push run on the requested tag. Diagnostic evidence never records publication.

Files and verification

  • .agents/scripts/full-loop-helper-state-lifecycle-commands.sh: scoped missing-run diagnostics.
  • .agents/scripts/tests/test-full-loop-completion-evidence.sh: five diagnostic cases using the existing CLI/receipt harness.
  • ShellCheck passed for both files; git diff --check passed; push-hook bun run typecheck passed.

Runtime Testing

Affected command-path cases passed: tag-push hint, wrong SHA, failed run, branch push, unavailable diagnostic API, explicit push publication and receipt preservation.

Blocker checkpoint

Full completion-evidence suite stops in its unchanged workflow-verification section with ModuleNotFoundError: No module named yaml. Python3 is available, but no worktree-local .venv/bin/python exists; tracked requirements.txt does not declare PyYAML and no .aidevops.json verification override is tracked. The legacy harness also resets PATH to non-Nix locations; a temporary invocation adapter preserved the existing tool PATH without repository changes. No dependency installation attempted.

Both implementation commits are pushed. PR remains draft; merge was not attempted because full local verification is incomplete. Recovery: rerun the existing completion-evidence suite on a provisioned runner with PyYAML and the declared CLI tools available, then complete readiness and one guarded merge attempt.


aidevops.sh v3.39.0 plugin for OpenCode v1.18.34 with gpt-6.1-sol spent 6m and 66,336 tokens on this as a headless worker.

@alex-solovyev alex-solovyev added the origin:worker Created by headless/pulse worker session label Oct 10, 2026
@coderabbitai

coderabbitai Bot commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)

🚫 Excluded labels (none allowed) (1)
  • no-review

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
  • Configuration used: Repository: marcusquinn/aidevops/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 05de0f99-e9d2-40fd-8b12-72ef951e1082

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
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@github-actions

github-actions Bot commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

Bash 3.2 Compatibility Regression Gate

✅ No regression — no new bash 3.2-incompatible construct(s).

Metric Base (038d414) Head (1d5a1f8)
Total bash 3.2-incompatible construct(s) 0 0

@github-actions

github-actions Bot commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

Shell Function Complexity Regression Gate

✅ No regression — no new function(s) >100 lines.

Metric Base (038d414) Head (1d5a1f8)
Total function(s) >100 lines 0 0

@github-actions

github-actions Bot commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

Qlty Smell Regression Gate

✅ No change — smell count unchanged.

Metric Base (038d414) Head (1d5a1f8) Delta
Total smells 65 65 +0

@alex-solovyev

alex-solovyev commented Oct 10, 2026 •

Copy link
Copy Markdown
Collaborator Author

Implemented exact-workflow/exact-SHA missing-release diagnostics and a matching tag-push retry hint without changing publication verification defaults. Added five focused regression cases. ShellCheck, affected command-path cases, and push-hook typecheck passed. Full local verification remains blocked by the runner missing Python yaml in the unchanged workflow section; this draft is a durable checkpoint, not ready for merge.


aidevops.sh v3.39.0 plugin for OpenCode v1.18.34 with gpt-6.1-sol spent 6m and 66,336 tokens on this as a headless worker.


Merged via PR #34290 to main.
Merged by deterministic merge pass (pulse-wrapper.sh).

@github-actions

github-actions Bot commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

Shell Nesting Depth Regression Gate

✅ No regression — no new file(s) with nesting depth >8.

Metric Base (038d414) Head (1d5a1f8)
Total file(s) with nesting depth >8 0 0

@github-actions

github-actions Bot commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

File Size Regression Gate

✅ No regression — no new non-README Markdown file(s) >1000 lines at root / >500 elsewhere.

Metric Base (038d414) Head (1d5a1f8)
Total non-README Markdown file(s) >1000 lines at root / >500 elsewhere 0 0

@github-actions

Copy link
Copy Markdown
Contributor

🔍 Code Quality Report

�[0;35m[MONITOR]�[0m Code Review Monitoring Report

SonarCloud: 0 bugs, 0 vulnerabilities, 1 code smells

Sat Oct 10 22:50:41 UTC 2026: Code review monitoring started
Sat Oct 10 22:50:41 UTC 2026: SonarCloud - Bugs: 0, Vulnerabilities: 0, Code Smells: 1
Sat Oct 10 22:53:16 UTC 2026: Codacy analysis completed - issues found (exit 2)

📈 Current Quality Metrics

  • BUGS: 0
  • CODE SMELLS: 1
  • VULNERABILITIES: 0

Generated on: Sat Oct 10 22:53:17 UTC 2026


Generated by AI DevOps Framework Code Review Monitoring

alex-solovyev and others added 3 commits October 11, 2026 03:00
…cile before blocker catch-up (#34291)

* GH#34287: fix(pulse): resume interrupted async housekeeping and reconcile before blocker catch-up

* GH#34287: fix(pulse): initialise housekeeping lock/state locals for unbound-var lint
…289) (#34292)

* fix(pulse): defer parent-close when reconcile budget is below mutation reserve (GH#34289)

* refactor: extract parent close postcondition helper to stay under complexity limit
…4286' into feature/auto-20261010-224140-gh34286
@marcusquinn
marcusquinn marked this pull request as ready for review October 10, 2026 23:31
@marcusquinn

Copy link
Copy Markdown
Owner

Maintainer verification — blocker resolved

The worker blocker was a missing local PyYAML for the full suite. That is an environment issue, not a code issue. I verified the PR on a host that has it:

  • Merged current main (c49f661737) into the branch, giving head 1d5a1f81e7.
  • bash .agents/scripts/tests/test-full-loop-completion-evidence.sh exits 0 with every case passing. This includes the new PASS missing release event suggests matching tag-push evidence without accepting mismatched or unavailable diagnostics case and the workflow-verification section that previously stopped on ModuleNotFoundError: yaml.
  • shellcheck -x reports no findings on both changed files.

Review: the change only adds diagnostics. Publication evidence is still derived solely from the release-event query. The --event push hint requires an exact SHA, tag head_branch, and a successful push run. Wrong-SHA, failed, branch-push and API-unavailable cases all omit the hint and record nothing.


aidevops.sh v3.39.1 plugin for OpenCode v1.18.35 with claude-opus-5-5

@github-actions

Copy link
Copy Markdown
Contributor

Markdown Lint (changed-line scoped)

No new violations in changed lines (0 pre-existing in touched files — ignored).

Metric Base (038d414) Head (1d5a1f8)
Total in touched files — 0
New (in changed lines) — 0

@github-actions

Copy link
Copy Markdown
Contributor

🔍 Code Quality Report

�[0;35m[MONITOR]�[0m Code Review Monitoring Report

SonarCloud: 0 bugs, 0 vulnerabilities, 1 code smells

Sat Oct 10 23:33:25 UTC 2026: Code review monitoring started
Sat Oct 10 23:33:25 UTC 2026: SonarCloud - Bugs: 0, Vulnerabilities: 0, Code Smells: 1
Sat Oct 10 23:35:53 UTC 2026: Codacy analysis completed - issues found (exit 2)

📈 Current Quality Metrics

  • BUGS: 0
  • CODE SMELLS: 1
  • VULNERABILITIES: 0

Generated on: Sat Oct 10 23:35:54 UTC 2026


Generated by AI DevOps Framework Code Review Monitoring

@alex-solovyev
alex-solovyev merged commit ec2fd36 into main Oct 10, 2026
56 of 57 checks passed
@alex-solovyev
alex-solovyev deleted the feature/auto-20261010-224140-gh34286 branch October 10, 2026 23:44
@github-actions github-actions Bot added the status:done Task is complete label Oct 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

origin:worker Created by headless/pulse worker session status:done Task is complete

Projects

None yet

Development

Successfully merging this pull request may close these issues.

record-published-release misses tag-push release runs unless --event push is given

2 participants