Repository navigation
chore: deploy skill library 4b65e21 (resolver reads the PR head) - #114
Merged
Merged
Conversation
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one): 1
Please check the settings in the CodeRabbit UI or the
 |
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Up to standards ✅🟢 Issues
|
There was a problem hiding this comment.
🟡 Changes recommended
A failed fetch can leave stale FETCH_HEAD data, causing both thread resolvers to post an off-PR SHA and resolve the thread incorrectly.
1 open issue
What changed in this PR
Deploys skill library 4b65e21 to WhoIsOnFirst’s development tooling. The substantive change directs review-thread resolvers to find fix commits on the fetched PR head rather than local HEAD. Application behavior and version remain unchanged.
Changes:
- Updates the Claude and Codex resolver instructions to fetch and inspect
pull/<prNumber>/head. - Synchronizes deployed library-version stamps to
4b65e212cbd7; all other changes are metadata-only.
| File | Description |
|---|---|
.codex/agents/pr-thread-resolver.toml |
Uses fetched PR history for fix citations; updates stamp. |
.codex/agents/pr-discover.toml |
Updates agent library stamp. |
.codex/agents/firecrawl-scraper.toml |
Updates agent library stamp. |
.claude/skills/wrap/SKILL.md |
Updates skill library stamp. |
.claude/skills/verification-before-completion/SKILL.md |
Updates skill library stamp. |
.claude/skills/vault-update/SKILL.md |
Updates skill library stamp. |
.claude/skills/start/SKILL.md |
Updates skill library stamp. |
.claude/skills/spec/SKILL.md |
Updates skill library stamp. |
.claude/skills/spec-tasks/SKILL.md |
Updates skill library stamp. |
.claude/skills/spec-review/SKILL.md |
Updates skill library stamp. |
.claude/skills/spec-requirements/SKILL.md |
Updates skill library stamp. |
.claude/skills/spec-reconcile/SKILL.md |
Updates skill library stamp. |
.claude/skills/spec-discovery/SKILL.md |
Updates skill library stamp. |
.claude/skills/spec-approach/SKILL.md |
Updates skill library stamp. |
.claude/skills/setup-foundation-context/SKILL.md |
Updates skill library stamp. |
.claude/skills/requesting-code-review/SKILL.md |
Updates skill library stamp. |
.claude/skills/release/SKILL.md |
Updates skill library stamp. |
.claude/skills/receiving-code-review/SKILL.md |
Updates skill library stamp. |
.claude/skills/pr-resolve/SKILL.md |
Updates skill library stamp. |
.claude/skills/pr-cleanup/SKILL.md |
Updates skill library stamp. |
.claude/skills/issue/SKILL.md |
Updates skill library stamp. |
.claude/skills/discover/SKILL.md |
Updates skill library stamp. |
.claude/skills/discernment-nudge/SKILL.md |
Updates skill library stamp. |
.claude/skills/context-drift/SKILL.md |
Updates skill library stamp. |
.claude/skills/codacy-code-review/SKILL.md |
Updates skill library stamp. |
.claude/skills/codacy-cloud-cli/SKILL.md |
Updates skill library stamp. |
.claude/skills/codacy-analysis-cli/SKILL.md |
Updates skill library stamp. |
.claude/skills/.library-manifest |
Records deployed library revision. |
.claude/rules/prose-core.md |
Updates rule library stamp. |
.claude/rules/code-core.md |
Updates rule library stamp. |
.claude/hooks/worktree-safety.py |
Updates stamp comment only. |
.claude/hooks/vale-staged.py |
Updates stamp comment only. |
.claude/hooks/docvault-lint.py |
Updates stamp comment only. |
.claude/hooks/block-pr-merge.py |
Updates stamp comment only. |
.claude/agents/pr-thread-resolver.md |
Uses fetched PR history for fix citations; updates stamp. |
.claude/agents/pr-discover.md |
Updates agent library stamp. |
.claude/agents/firecrawl-scraper.md |
Updates agent library stamp. |
.agents/skills/wrap/SKILL.md |
Updates skill library stamp. |
.agents/skills/verification-before-completion/SKILL.md |
Updates skill library stamp. |
.agents/skills/vault-update/SKILL.md |
Updates skill library stamp. |
.agents/skills/start/SKILL.md |
Updates skill library stamp. |
.agents/skills/spec/SKILL.md |
Updates skill library stamp. |
.agents/skills/spec-tasks/SKILL.md |
Updates skill library stamp. |
.agents/skills/spec-review/SKILL.md |
Updates skill library stamp. |
.agents/skills/spec-requirements/SKILL.md |
Updates skill library stamp. |
.agents/skills/spec-reconcile/SKILL.md |
Updates skill library stamp. |
.agents/skills/spec-discovery/SKILL.md |
Updates skill library stamp. |
.agents/skills/spec-approach/SKILL.md |
Updates skill library stamp. |
.agents/skills/setup-foundation-context/SKILL.md |
Updates skill library stamp. |
.agents/skills/requesting-code-review/SKILL.md |
Updates skill library stamp. |
.agents/skills/release/SKILL.md |
Updates skill library stamp. |
.agents/skills/receiving-code-review/SKILL.md |
Updates skill library stamp. |
.agents/skills/pr-resolve/SKILL.md |
Updates skill library stamp. |
.agents/skills/pr-cleanup/SKILL.md |
Updates skill library stamp. |
.agents/skills/issue/SKILL.md |
Updates skill library stamp. |
.agents/skills/discover/SKILL.md |
Updates skill library stamp. |
.agents/skills/discernment-nudge/SKILL.md |
Updates skill library stamp. |
.agents/skills/context-drift/SKILL.md |
Updates skill library stamp. |
.agents/skills/code-review/SKILL.md |
Updates skill library stamp. |
.agents/skills/codacy-code-review/SKILL.md |
Updates skill library stamp. |
.agents/skills/codacy-cloud-cli/SKILL.md |
Updates skill library stamp. |
.agents/skills/codacy-analysis-cli/SKILL.md |
Updates skill library stamp. |
.agents/rules/prose-core.md |
Updates rule library stamp. |
.agents/rules/code-core.md |
Updates rule library stamp. |
Review effort: Balanced
Comment on lines
+90
to
+91
| git fetch -q origin "pull/<prNumber>/head" | ||
| git log --oneline --max-count=5 FETCH_HEAD -- <file> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Deploys skill library
4b65e21. Follows DEVS-151 and the review on lbruton/SessionFlow#139.pr-thread-resolver: a "Resolved" reply lists fix commits from the PR head (pull/<prNumber>/head), not localHEAD, so every SHA it can cite is on the PR.library-versionstamps in the deployed rules, skills, agents and hooks.No version bump: this is a
chorePR.