Skip to content

Support direct in-chat planning - #39

Merged
chrisbanes merged 2 commits into
mainfrom
agent/direct-in-chat-to-plan
Aug 16, 2026
Merged

Support direct in-chat planning#39
chrisbanes merged 2 commits into
mainfrom
agent/direct-in-chat-to-plan

Conversation

@chrisbanes

Copy link
Copy Markdown
Owner

Summary

  • make /to-plan conduct its own in-chat interview for inline tasks or prior conversation context
  • continue from explicit summary confirmation to the existing scratch-plan handoff without mode changes or reinvocation
  • preserve GitHub issue mode, GitHub-only --auto, and all existing GitHub gates
  • prevent an inline task from accidentally reusing an unrelated earlier confirmed summary

Why

Conversation planning previously required a completed grill-me session and a second /to-plan invocation. Inline task precedence also needed an explicit source-selection rule to avoid treating a prior task's confirmed summary as the current task.

Validation

  • python3 /Users/chris/.codex/skills/.system/skill-creator/scripts/quick_validate.py skills/to-plan
  • npm run lint
  • git diff --check origin/main...HEAD

@chrisbanes
chrisbanes marked this pull request as ready for review August 16, 2026 09:01
@greptile-apps

greptile-apps Bot commented Aug 16, 2026

Copy link
Copy Markdown

Greptile Summary

The PR enables /to-plan to interview users and continue planning within the same invocation while preserving the existing GitHub workflow.

  • Defines precedence and clarification rules for inline tasks and prior confirmed summaries.
  • Continues from confirmed conversation summaries into scratch-plan generation without reinvocation.
  • Updates routing documentation and OpenAI metadata to describe in-chat planning.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
README.md Updates the workflow overview to advertise in-chat tasks as supported planning inputs.
skills/to-plan/SKILL.md Adds direct interview and same-invocation planning behavior, with explicit clarification when multiple prior summaries could apply.
skills/to-plan/agents/openai.yaml Updates the default prompt to describe in-chat task support.
skills/using-chrisbanes-skills/SKILL.md Updates skill routing guidance so in-chat planning requests select to-plan.

Reviews (2): Last reviewed commit: "Clarify in-chat plan source selection" | Re-trigger Greptile

Comment thread skills/to-plan/SKILL.md Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 12780459f1

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread skills/to-plan/SKILL.md Outdated
@chrisbanes
chrisbanes merged commit 5b8c4b0 into main Aug 16, 2026
2 checks passed
@chrisbanes
chrisbanes deleted the agent/direct-in-chat-to-plan branch August 16, 2026 09:14
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