Skip to content

Add gen-idea skill to Codex installs - #101

Open
zhaoxueyan1 wants to merge 3 commits into
PolyArch:devfrom
zhaoxueyan1:codex/gen-idea-codex-skill
Open

Add gen-idea skill to Codex installs#101
zhaoxueyan1 wants to merge 3 commits into
PolyArch:devfrom
zhaoxueyan1:codex/gen-idea-codex-skill

Conversation

@zhaoxueyan1

Copy link
Copy Markdown

Summary

  • add a humanize-gen-idea skill so Codex and Kimi installs expose the existing gen-idea workflow
  • update the shared installer and install docs to sync and document the new skill
  • add wiring regression coverage for gen-idea skill installation and include it in the test runner

Test Plan

  • bash tests/test-gen-idea-wiring.sh
  • bash tests/test-codex-hook-install.sh

CopilotAI review requested due to automatic review settings April 21, 2026 07:15

CopilotAI 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.

Pull request overview

This PR exposes the existing gen-idea workflow to Codex/Kimi installs by adding a new humanize-gen-idea entrypoint skill, updating the unified installer + install docs, and adding regression coverage to ensure the wiring stays intact.

Changes:

  • Add skills/humanize-gen-idea flow skill and include it in the unified installer’s SKILL_NAMES.
  • Update Codex/Kimi/Claude install docs to mention the new skill/command.
  • Add wiring + install regression tests and include them in the test runner.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
FileDescription
skills/humanize-gen-idea/SKILL.mdNew flow skill that documents the required gen-idea execution sequence for Codex/Kimi entrypoints.
scripts/install-skill.shAdds humanize-gen-idea to the install set and generalizes installed-layout detection to iterate SKILL_NAMES.
docs/install-for-codex.mdDocuments that Codex installs now include the humanize-gen-idea skill directory.
docs/install-for-kimi.mdDocuments that Kimi installs now include the humanize-gen-idea skill and updates manual install steps accordingly.
docs/install-for-claude.mdAdds /humanize:gen-idea to the list of post-install commands.
tests/test-gen-idea-wiring.shNew test to assert skill + installer + docs wiring for gen-idea.
tests/test-codex-hook-install.shExtends Codex install regression to assert humanize-gen-idea is installed.
tests/run-all-tests.shAdds the new wiring test to the test suite list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment threaddocs/install-for-kimi.md

@chatgpt-codex-connectorchatgpt-codex-connectorBot 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:9671b42681

ℹ️ 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 threadscripts/install-skill.sh
@SihaoLiu

Copy link
Copy Markdown
Contributor

@shinezyy can you take a look?

@SihaoLiu
SihaoLiu requested a review from shinezyyApril 22, 2026 22:21
@zevorn

Copy link
Copy Markdown
Contributor

@zhaoxueyan1 Please resolve conficts

@zhaoxueyan1

Copy link
Copy Markdown
Author

@zevorn Resolved.

Sign up for freeto 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.

4 participants

@zhaoxueyan1@SihaoLiu@zevorn