Commit Graph

244 Commits

Author SHA1 Message Date
toyota 1306f528b4 fix: use correct JSON types in pr-triage output format examples
Fix Major review findings: comment_id, line, pr_number should be
number type, and incremental, has_*_changes, carried_over should
be boolean type instead of strings.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 21:17:22 +09:00
toyota de7acc77f7 feat(ci): add OpenTelemetry metrics to triage step
Enable OTEL telemetry export for the Haiku triage job to collect
token usage and performance metrics via Grafana.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 21:10:25 +09:00
toyota 453845c9b3 fix(ci): revert to simple prompt now that hidden file upload is fixed
Remove Bash heredoc workaround instructions and diagnostic step.
The root cause was upload-artifact excluding dot-prefixed files,
not the Write tool. Revert allowedTools to original Bash(gh:*).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 21:09:04 +09:00
toyota 9b8b14b045 fix(ci): enable include-hidden-files for artifact upload
upload-artifact@v4 defaults include-hidden-files to false, which
excludes dot-prefixed files like .pr-triage.json. Enable it explicitly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 21:02:51 +09:00
toyota 6e2b9b41fb fix(ci): add file verification step before artifact upload
Add diagnostic step to check .pr-triage.json existence and location
before upload-artifact, to isolate whether the issue is file creation
or file discovery.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 21:00:17 +09:00
toyota 031905de21 fix(ci): allow cat/test commands in triage allowedTools
Bash(gh:*) was blocking cat (heredoc for file creation) and test
(file existence check) commands, preventing .pr-triage.json from
being created.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 20:57:06 +09:00
toyota ed60814ea0 fix(ci): use Bash heredoc instead of Write tool for .pr-triage.json output
Write tool output may not persist after claude-code-action SDK execution.
Use Bash heredoc to write directly to the filesystem.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 20:51:34 +09:00
toyota 4235b05568 fix(ci): skip full diff for large PRs (>= 15 files) to prevent context overflow
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 20:44:48 +09:00
toyota 21cb264d29 fix(ci): optimize triage to avoid full diff for large PRs
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 20:34:26 +09:00
toyota be84e97c9f fix(ci): increase triage max-turns to 20 for full triage fallback
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 20:30:04 +09:00
toyota e1784459ad fix(ci): add show_full_output to triage, increase max-turns to 15, error on missing artifact
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 20:23:54 +09:00
toyota f7b35959ad fix(ci): correct Haiku model ID to claude-haiku-4-5@20251001
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 20:19:03 +09:00
toyota 593b6296a9 chore: bump development-workflows version to 0.2.0
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 20:05:53 +09:00
toyota 14b024c62b feat(ci): prepare triage job for pr-triage skill migration (commented out until merge)
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 20:05:53 +09:00
toyota f46cba813c feat(pr-triage): add pr-triage skill for CI triage phase
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 20:05:53 +09:00
Yoichi Toyota 6865ba2b54 Apply suggestions from code review
Co-authored-by: xtone-claude-code-action[bot] <215095484+xtone-claude-code-action[bot]@users.noreply.github.com>
2026-03-06 19:44:09 +09:00
toyota 9f86f14686 feat(ci): add review state cache, skills install, and Opus model to PR review workflow
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 19:36:52 +09:00
toyota 98be695a23 chore: remove .claude/skills/claude-skill-review (merged into code-review)
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 19:33:24 +09:00
toyota fd66ff806e refactor(ci): replace /claude-skill-review with /code-review in PR review workflow
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 19:32:27 +09:00
toyota d266633e7a feat(code-review): integrate SKILL.md review into code-review workflow
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 19:24:51 +09:00
toyota 8ab8e711d5 feat(code-review): add skill review reference files
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 19:24:37 +09:00
Yoichi Toyota 6a1c636630 Merge pull request #111 from xtone/feature/code-review-incremental-review
feat(code-review): PR更新時のインクリメンタルレビュー対応
2026-03-06 10:49:08 +09:00
TOYOTA, Yoichi a0e6c53486 refactor(code-review): address PR #111 review comments
- Separate runtime files from directory tree into dedicated section
- Extract incremental review details to references/incremental-review.md
- Add error handling note for corrupted .pr-review-state.json

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 09:55:44 +09:00
TOYOTA, Yoichi a95af5a234 chore(code-review): bump version to 1.2.0
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 09:24:01 +09:00
TOYOTA, Yoichi fe179c8044 feat(code-review): add incremental review for PR updates with state caching
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 09:19:55 +09:00
Yoichi Toyota 4066f1e359 Merge pull request #110 from xtone/feature/code-review-cost-optimization
feat(code-review): Haiku+Opus 2ジョブ構成によるCI実行コスト最適化
2026-03-05 20:40:14 +09:00
toyota 6418170172 fix(code-review): add TOC to ci-optimized-workflow.md
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-05 20:36:27 +09:00
Yoichi Toyota e7cbcbd7ca Merge pull request #109 from xtone/feature/add_otel_env_for_claude
Configure OpenTelemetry settings in PR workflow
2026-03-05 20:29:34 +09:00
toyota 35cfcff72c chore(code-review): bump version to 1.1.0
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-05 20:28:53 +09:00
toyota 8fd3075bef feat(code-review): add Haiku+Opus two-job CI workflow for cost optimization
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-05 20:28:43 +09:00
Yoichi Toyota 1383bb753e Configure OpenTelemetry settings in PR workflow
Add OpenTelemetry configuration for metrics and logs.
2026-03-05 20:10:17 +09:00
Pregum 21ffd3e4bd Merge pull request #70 from xtone/feature/skill-tracker-jsonl-export
feat(skill-usage-tracker): Notion MCP直接呼び出しに変更
2026-03-03 16:49:14 +09:00
Pregum 285f6f5239 fix(skill-usage-tracker): skill_usage_sender.jsにJSONパースの行単位エラーハンドリングを追加
PRレビュー指摘対応。loadEvents()でJSONL各行のパース失敗時にクラッシュせず
静かにスキップするよう修正(skill-stats.jsと同様のパターン)。

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-03 16:45:20 +09:00
masaya ishihara 51913bb0ec Merge pull request #107 from xtone/feature/rules-merge-skip-triad
feat: limit /rules-merge to 01-89 range, skip triad files
2026-03-03 14:32:22 +09:00
masaya ishihara 9dba150f55 feat: limit /rules-merge to 01-89 range, skip triad files (90+)
90番台は .claude/rules/ から自動ロードされるため、CLAUDE.mdに
含めるとコンテキストが重複する。マージ対象を01-89番台に限定し、
末尾に90番台の参照注記を付与する。

- /rules-merge: 01-89番台のみ連結、90番台スキップ + 末尾参照注記
- /rules-status: マージ対象/自動ロードを区別して表示
- 重複チェック: CLAUDE.md + 対象分割ファイルの両方を参照
- /rules-init 実行例を更新

Closes #106

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-03 14:29:15 +09:00
masaya ishihara bc48a4f609 Merge pull request #105 from xtone/feature/add-rules-status-command
feat: add /rules-status command and /rules-merge diff summary
2026-03-03 13:47:37 +09:00
masaya ishihara 7a783c53f8 feat: add /rules-status command and /rules-merge diff summary
- Add /rules-status subcommand for file listing, line counts, and
  health checks (duplicate numbers, zone mixing, empty files)
- Add diff summary to /rules-merge step 3 showing changes since
  last CLAUDE.md generation
- Bump version to v1.1

Closes #103, Closes #104

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-03 13:43:52 +09:00
masaya ishihara a4e22bd9a6 Merge pull request #102 from xtone/feature/add-rules-split-manager-skill
feat: add rules-split-manager skill
2026-03-03 10:43:20 +09:00
masaya ishihara 28a4c9114b fix: address PR review feedback for rules-split-manager skill
- Add trigger condition to description (when to use)
- Add concrete input/output example for /rules-init
- Remove date from version info

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-03 10:40:28 +09:00
masaya ishihara 18360bbe94 feat: add rules-split-manager skill for CLAUDE.md split management
Manage CLAUDE.md via split rule files in .claude/rules/. Redirects
learning triad (/lessons, /review-learn, /ci-learn) writes to
numbered split files and auto-merges them back into CLAUDE.md.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-03 10:27:24 +09:00
masaya ishihara e62ce9ceea Merge pull request #101 from xtone/feature/add-crash-analyzer-skill
feat: add crash-analyze skill for Firebase Crashlytics triage
2026-02-25 11:54:35 +09:00
masaya ishihara c3dd877a8f fix: use passive voice in description trigger keywords
Change "以下のキーワードで起動:" to "〜で使用される" for
consistent third-person passive voice per review feedback.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-25 11:45:26 +09:00
masaya ishihara 1aa1829fa4 fix: use third-person trigger description in skill frontmatter
Change "以下の場合に使用:" to "以下のキーワードで起動:" per review
feedback to maintain consistent third-person tone.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-25 11:38:50 +09:00
masaya ishihara fcbe1d6c3c fix: use third-person description in skill frontmatter
Address review feedback: change description from imperative form
(〜してください) to third-person declarative form (〜するスキル).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-25 11:32:10 +09:00
masaya ishihara fed63791b2 feat: add crash-analyze skill for Firebase Crashlytics triage
Add Android-specific skill that analyzes stack traces from Firebase
Crashlytics and provides triage classification (MONITOR/INVESTIGATE/
FIX/DEFEND) with actionable fix suggestions.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-25 11:25:09 +09:00
Pregum 4be865fecb Merge pull request #55 from xtone/feature/add-flutter-golden-test-skill
feat: Flutter Golden Test スキルの追加
2026-02-17 17:34:21 +09:00
masaya ishihara a630536ee4 Merge pull request #100 from xtone/feature/add-ci-learner-skill
feat: add ci-learner skill for CI failure pattern learning
2026-02-17 10:12:58 +09:00
masaya ishihara b381457b32 feat: add ci-learner skill for CI failure pattern learning
Third pillar of the learning triad (lessons/review-learn/ci-learn).
Analyzes CI/GitHub Actions failure logs and accumulates patterns
into CLAUDE.md to prevent recurring CI failures.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-17 09:52:06 +09:00
masaya ishihara 5debb90c34 Merge pull request #99 from xtone/fix/video-keyframe-analyzer-skill-discovery
fix: video-keyframe-analyzer がセッション再起動で認識されない問題を修正
2026-02-16 12:44:45 +09:00
masaya ishihara dd6e677b40 fix: quote SKILL.md description to fix YAML parsing failure
video-keyframe-analyzerのSKILL.mdフロントマターのdescriptionが
クォートなしで$, %, 括弧等の特殊文字を含んでおり、YAMLパース失敗で
スキルが認識されない問題を修正。他スキルと同様にダブルクォートで囲む。

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-16 12:33:43 +09:00