Skip to content

fix(ci): v4 対応 — test-core を test-go に置換 + CHANGELOG compare link 追加 #71

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Chachamaru127 merged 6 commits into from
Apr 12, 2026
Merged

fix(ci): v4 対応 — test-core を test-go に置換 + CHANGELOG compare link 追加 #71

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
4a01650
fix(ci): v4 対応 — test-core を test-go に置換 + CHANGELOG compare link 追加
Apr 12, 2026
e599741
fix(sync): harness binary 不在時も cache sync を完走させる — CI 対応
Apr 12, 2026
94b2ed9
fix(mirror): skills/ci と skills/deploy のミラー drift を解消
Apr 12, 2026
41c80e4
fix(opencode): build-opencode.js を実行して opencode/ を同期
Apr 12, 2026
6f597b0
fix(skills): 配布対象外の IMPLEMENTATION_SUMMARY.md を skills/ から削除
Apr 12, 2026
82f3080
fix(ci): internal-only skill を codex mirror drift check から除外
Apr 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .github/workflows/opencode-compat.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,10 @@ jobs:
diff -rq skills/ codex/.codex/skills/ 2>/dev/null \
| grep -v ".DS_Store" \
| grep -v "^Only in skills/: cc-update-review$" \
| grep -v "^Only in skills/: allow1$" \
| grep -v "^Only in skills/: claude-codex-upstream-update$" \
| grep -v "^Only in skills/: x-announce$" \
| grep -v "^Only in skills/: x-article$" \
|| true
)
if [ -n "$DIFF" ]; then
Expand Down
29 changes: 14 additions & 15 deletions .github/workflows/validate-plugin.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,29 +39,28 @@ jobs:
run: |
bash ./tests/test-codex-package.sh

test-core:
test-go:
runs-on: ubuntu-latest
timeout-minutes: 10

steps:
- name: Checkout
uses: actions/checkout@v4

- name: Setup Node.js
uses: actions/setup-node@v4
- name: Setup Go
uses: actions/setup-go@v5
with:
node-version: "20"
cache: "npm"
cache-dependency-path: core/package.json
go-version-file: go/go.mod
cache-dependency-path: go/go.sum

- name: Install core dependencies
working-directory: core
run: npm install --package-lock=false
- name: Build harness binary
working-directory: go
run: go build ./cmd/harness/

- name: Typecheck core
working-directory: core
run: npm run typecheck
- name: Run Go tests
working-directory: go
run: go test ./...

- name: Run vitest
working-directory: core
run: npm test
- name: Run go vet
working-directory: go
run: go vet ./...
Comment on lines +42 to +66

@coderabbitai coderabbitai Bot Apr 12, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🔴 Critical

ワークフローファイル変更はこのリポジトリ規約違反です。

この PR で .github/workflows/validate-plugin.yml を直接変更しており、リポジトリの保護ルールに抵触します。CI 修正方針自体は理解できますが、このファイル変更は差し戻し対象です。

As per coding guidelines: .github/workflows/*: Deny modifications to .github/workflows/* to protect CI/CD pipeline integrity

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In @.github/workflows/validate-plugin.yml around lines 42 - 66, This change
modifies the protected CI workflow .github/workflows/validate-plugin.yml (the
test-go job with steps like "Setup Go", "Build harness binary", "Run Go tests",
"Run go vet"), which violates repository rules; revert the edits to that file
and remove any modifications to the test-go job, then propose the CI adjustments
via the approved process (open an issue or a separate PR requesting maintainers
to update .github/workflows/validate-plugin.yml) or move non-CI changes into a
different location (e.g., scripts/ or docs/) so no direct changes are made to
the protected workflow file.

4 changes: 3 additions & 1 deletion CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2894,7 +2894,9 @@ Purpose: 自己修正ループ失敗時に「止まるだけ」から「次の

For v2.9.x and earlier, see [GitHub Releases](https://github.com/Chachamaru127/claude-code-harness/releases).

[Unreleased]: https://github.com/Chachamaru127/claude-code-harness/compare/v4.0.0...HEAD
[Unreleased]: https://github.com/Chachamaru127/claude-code-harness/compare/v4.0.2...HEAD
[4.0.2]: https://github.com/Chachamaru127/claude-code-harness/compare/v4.0.1...v4.0.2
[4.0.1]: https://github.com/Chachamaru127/claude-code-harness/compare/v4.0.0...v4.0.1
[4.0.0]: https://github.com/Chachamaru127/claude-code-harness/compare/v3.17.1...v4.0.0
[3.17.1]: https://github.com/Chachamaru127/claude-code-harness/compare/v3.17.0...v3.17.1
[3.17.0]: https://github.com/Chachamaru127/claude-code-harness/compare/v3.16.0...v3.17.0
Expand Down
2 changes: 1 addition & 1 deletion codex/.codex/skills/ci/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name: ci
description: "CIが赤くなったら呼んで。パイプライン消防隊、出動します。Use when user mentions CI failures, build errors, test failures, or pipeline issues. Do NOT load for: local builds, standard implementation work, reviews, or setup."
description-en: "CI red? Call us. Pipeline fire brigade deploys. Use when user mentions CI failures, build errors, test failures, or pipeline issues. Do NOT load for: local builds, standard implementation work, reviews, or setup."
description-ja: "CIが赤くなったら呼んで。パイプライン消防隊、出動します。Use when user mentions CI failures, build errors, test failures, or pipeline issues. Do NOT load for: local builds, standard implementation work, reviews, or setup."
allowed-tools: ["Read", "Grep", "Bash", "Task"]
allowed-tools: ["Read", "Grep", "Bash", "Task", "Monitor"]
user-invocable: false
context: fork
argument-hint: "[analyze|fix|run]"
Expand Down
2 changes: 1 addition & 1 deletion codex/.codex/skills/deploy/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name: deploy
description: "VercelやNetlifyへいざ出陣。本番環境への片道切符を手配します。Use when user mentions deployment, Vercel, Netlify, analytics, or health checks. Do NOT load for: implementation work, local development, reviews, or setup."
description-en: "Deploy to Vercel/Netlify. One-way ticket to production arranged. Use when user mentions deployment, Vercel, Netlify, analytics, or health checks. Do NOT load for: implementation work, local development, reviews, or setup."
description-ja: "VercelやNetlifyへいざ出陣。本番環境への片道切符を手配します。Use when user mentions deployment, Vercel, Netlify, analytics, or health checks. Do NOT load for: implementation work, local development, reviews, or setup."
allowed-tools: ["Read", "Write", "Edit", "Bash"]
allowed-tools: ["Read", "Write", "Edit", "Bash", "Monitor"]
disable-model-invocation: true
user-invocable: false
argument-hint: "[vercel|netlify|health]"
Expand Down
269 changes: 0 additions & 269 deletions codex/.codex/skills/generate-video/schemas/IMPLEMENTATION_SUMMARY.md
View file
Open in desktop

This file was deleted.

Loading
Loading