fix(test): align issue workflow reference assertion

[YiWang24]

Summary

• Update tests/actions/self-test-routing.bats assertion for issue-ops.yml to match the current reusable workflow reference format.
• Validate the owner/repo path plus 40-char SHA pin in the uses: value.

Verification

• bats tests/ --recursive

Artifacts

• Conversation: https://app.warp.dev/conversation/e93f8a46-c4a0-436b-b344-fc94fd5eaa8d

Co-Authored-By: Oz oz-agent@warp.dev

---

^{Need help on this PR? Tag @codesmith with what you need.}

• Let Codesmith autofix CI failures and bot reviews

Summary by CodeRabbit

• Tests
  • Updated workflow validation to enforce fully pinned reusable workflow references with commit SHA instead of relative paths, ensuring stricter version control for workflow dependencies.

[qodo-code-review]

ⓘ You've reached your Qodo monthly free-tier limit. Reviews pause until next month — upgrade your plan to continue now, or link your paid account if you already have one.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

A test assertion in the issue-routing test suite was updated to require a fully SHA-pinned external reusable-workflow reference (YiAgent/OpenCI/.github/workflows/reusable/issue.yml@<40-hex>) instead of accepting the unpinned local path.

Changes

Test Assertion Update

Layer / File(s)	Summary
Test Expectation tests/actions/self-test-routing.bats	The issue-ops.yml calls reusable issue.yml assertion now requires a fully pinned uses: reference with a 40-character SHA hash instead of a local unpinned path.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• test: add 424 comprehensive tests for issue and PR workflows #33: Originally introduced the unpinned uses: ./.github/workflows/reusable/issue.yml assertion that this PR now tightens to require full SHA pinning.
• fix(workflows): replace relative reusable paths with full org/repo refs #35: Related changes that enforce full org/repo@SHA pins for reusable-workflow references across tests and workflows.
• fix(claude-harness): vendor OpenCI into .openci so verify-sha-consistency passes #3: Explored an alternative approach using vendored OpenCI workflows with local path resolution, rather than enforcing external SHA-pinned references.

Poem

🐰 A hop and a pin, the workflow's been locked,
No more local paths that drift and unblock—
With SHA in tow, the reference runs true,
External and trusted, your CI shines through! ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: aligning an issue workflow reference assertion in tests, which matches the changeset's focus on updating the self-test-routing.bats file.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/self-test-routing-issue-uses-assertion

---

_{Review rate limit: 9/10 reviews remaining, refill in 6 minutes.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud