Skip to content

add claude review agent - #299

Merged
jtdub merged 2 commits into
masterfrom
claude-review
Aug 23, 2026
Merged

add claude review agent#299
jtdub merged 2 commits into
masterfrom
claude-review

Conversation

@jtdub

@jtdub jtdub commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Summary

Self-Review Checklist

  • poetry run ./scripts/build.py lint-and-test passes locally (lint + 95% test coverage).
  • Tests were written first (TDD) and cover the change, following the testing conventions.
  • CHANGELOG.md has an entry under ## [Unreleased] referencing this issue/PR ((#NNN)).
  • Documentation is updated if public API or driver behavior changed (and mkdocs build --strict passes if docs were touched).
  • Commit messages follow the contributing guide: imperative mood, subject ≤72 characters, body explains why.

AI-Assisted Contributions

If this PR was written with an AI coding tool, review it against the repo standards before requesting review: Claude Code users can run the hier-config-review skill; other tools should be pointed at AGENTS.md and the developer docs.

@jtdub
jtdub requested a review from aedwardstx as a code owner August 23, 2026 14:19
claude-code-action exchanges a GitHub OIDC token for its App token. The
job did not request id-token: write, so the runner supplied no OIDC
endpoint and the action failed with "Unable to get
ACTIONS_ID_TOKEN_REQUEST_URL env variable" after three attempts.

Claude-Session: https://claude.ai/code/session_01HbMucj2WqnL685BRUcFhZT
@jtdub

jtdub commented Aug 23, 2026

Copy link
Copy Markdown
Contributor Author

@claude review

@jtdub
jtdub merged commit 98a9a49 into master Aug 23, 2026
7 checks passed
@jtdub
jtdub deleted the claude-review branch August 23, 2026 14:27
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