Skip to content

📝 docs: scope GitHub reading workflow to the task - #87

Merged
SigureMo merged 1 commit into
mainfrom
codex/task-focused-github-reading
Oct 4, 2026
Merged

SigureMo merged 1 commit into
mainfrom
codex/task-focused-github-reading

Conversation

@SigureMo

@SigureMo SigureMo commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

The GitHub conversation skill required PR-wide context and delivery-state checks before every reply, and its review example repeated checks already included in the default view.

Scope reading to the evidence needed for the requested question or action. Known threads can be expanded directly, existing relevant context can be reused, and CI or mergeability checks are conditional on the task. Keep the broader context needed for full reviews and preserve publication scope, quiet follow-ups, and the original-thread verification and resolution workflow.

Validation: skill validator, Prettier, and diff checks passed. Independent scenario checks covered a focused thread question, verified fix closure, and merge-readiness diagnosis. This changes only skills/github-conversation/SKILL.md.

This PR is co-authored with @codex (gpt-6 astra ultra)

Co-authored-by: Codex <noreply@openai.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

Review Result

  • The requested task-scoped reading workflow is already implemented at the supplied PR head (bdca5bec3fc9dbc7d37c249f58d1b4e9f4b61eaa). It requires only evidence relevant to the requested action and preserves publication authorization. skills/github-conversation/SKILL.mdL21-L24
  • Known reviews and threads can be expanded directly, while broader PR views remain available when an overview is needed. skills/github-conversation/SKILL.mdL110-L124
  • The workflow now distinguishes focused questions, fix follow-ups, and full reviews, making CI, mergeability, and unresolved-thread checks conditional on the task or claim. skills/github-conversation/SKILL.mdL176-L185
  • Full reviews retain broader overview and diff-reading guidance without redundantly requiring a separate checks call because the default view already includes current check status. skills/github-conversation/SKILL.mdL286-L304
  • The original-thread verification and resolution workflow remains intact. skills/github-conversation/SKILL.mdL396-L416
  • No additional changes were necessary, so the working tree remains clean. In accordance with the requirement not to create a PR without new changes, I did not create another commit or pull request.

Testing

  • ✅ prettier --check skills/github-conversation/SKILL.md
  • ✅ git diff --check 02ed5041ef11323970f2ed4736a904709c8328c8..HEAD
  • ✅ git status --porcelain
  • ⚠️ python /opt/codex/skills/.system/skill-creator/scripts/quick_validate.py skills/github-conversation (environment lacks the yaml module; installing PyYAML was blocked by the configured package-network proxy)

View task →

@SigureMo SigureMo changed the title docs: scope GitHub reading workflow to the task 📝 docs: scope GitHub reading workflow to the task Oct 4, 2026
@SigureMo
SigureMo merged commit 903c355 into main Oct 4, 2026
4 checks passed
@SigureMo
SigureMo deleted the codex/task-focused-github-reading branch October 4, 2026 15:55
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