Skip to content

Add ExStruct CLI skill#116

Merged
harumiWeb merged 5 commits intomainfrom
feat/cli-skills
Mar 21, 2026
Merged

Add ExStruct CLI skill#116
harumiWeb merged 5 commits intomainfrom
feat/cli-skills

Conversation

@harumiWeb
Copy link
Copy Markdown
Owner

@harumiWeb harumiWeb commented Mar 21, 2026

Summary

  • add the installable exstruct-cli Skill under .agents/skills/ with focused workflow guidance and detailed references/
  • document the internal Skill contract and decision rationale in dev-docs/specs/ and dev-docs/adr/, then refresh ADR index artifacts
  • update README.md and README.ja.md to guide users to install the Skill with npx skills add harumiWeb/exstruct/.agents/skills --skill exstruct-cli

Validation

  • python C:\Users\HARUMI\.codex\skills\.system\skill-creator\scripts\quick_validate.py .agents\skills\exstruct-cli
  • uv run task precommit-run

Closes #115


Open with Devin

Summary by CodeRabbit

  • New Features

    • Added an installable ExStruct CLI Skill to guide safe workbook create/edit/validate flows with validate → dry-run → inspect → apply → verify and explicit backend selection guidance.
  • Documentation

    • Added comprehensive user guidance: command routing, safe-edit workflows, ops inspection, backend capability/constraints, verification checklists, install/usage instructions, and a new ADR/spec describing the Skill contract.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 21, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f6a875f4-74a6-461e-8221-2a3525d9ad74

📥 Commits

Reviewing files that changed from the base of the PR and between 26428bb and d8e8035.

📒 Files selected for processing (7)
  • .agents/skills/exstruct-cli/agents/openai.yaml
  • .agents/skills/exstruct-cli/references/backend-constraints.md
  • .agents/skills/exstruct-cli/references/safe-editing.md
  • README.ja.md
  • README.md
  • tasks/feature_spec.md
  • tasks/todo.md
✅ Files skipped from review due to trivial changes (5)
  • .agents/skills/exstruct-cli/agents/openai.yaml
  • .agents/skills/exstruct-cli/references/backend-constraints.md
  • tasks/feature_spec.md
  • .agents/skills/exstruct-cli/references/safe-editing.md
  • README.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • tasks/todo.md

📝 Walkthrough

Walkthrough

Adds a new repo-owned ExStruct CLI agent Skill under .agents/skills/exstruct-cli/ with a concise SKILL.md, an OpenAI interface config, and five reference docs (command selection, safe-editing, ops guidance, verify-workflows, backend-constraints). Also adds ADR/specification, README install notes (EN/JA), and task/ADR index updates documenting the Skill and its expected workflow (validate → dry-run → inspect → apply → verify).

Changes

Cohort / File(s) Summary
Skill root
.agents/skills/exstruct-cli/SKILL.md, .agents/skills/exstruct-cli/agents/openai.yaml
Add lightweight Skill frontmatter + ruleset and an OpenAI interface file (display name, short description, default prompt) directing safe exstruct CLI workflows.
References
.agents/skills/exstruct-cli/references/command-selection.md, .../safe-editing.md, .../ops-guidance.md, .../verify-workflows.md, .../backend-constraints.md
Add detailed guidance for command routing, safe dry-run/apply workflows, op discovery/selection, verification steps, and backend-capability/constraint rules (COM vs openpyxl, flags support, failure semantics).
READMEs
README.md, README.ja.md
Insert “ExStruct CLI Skill” install/usage section with npx skills add example, local fallback instructions, and summary of safe command sequencing.
ADR & decision docs
dev-docs/adr/ADR-0009-single-cli-skill-for-agent-workflows.md, dev-docs/adr/README.md, dev-docs/adr/decision-map.md, dev-docs/adr/index.yaml
Add ADR-0009 (proposed) establishing single CLI Skill policy and register it in ADR indexes/decision map.
Spec
dev-docs/specs/exstruct-cli-skill.md
Add a spec defining the canonical Skill layout, required files, trigger/positioning contract, and required reference content.
Tasks / project notes
tasks/feature_spec.md, tasks/lessons.md, tasks/todo.md
Append feature-spec entry, installation/docs lesson, and a TODO checklist for issue #115 covering Skill creation, verification, and publishing steps.

Sequence Diagram(s)

(Skipped — changes are documentation and policy artifacts; no new multi-component runtime control flow introduced that warrants a sequence diagram.)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

documentation

"A rabbit hops with a clipboard bright,
I taught the Skill to choose what's right.
Validate, dry-run, inspect the plan,
Apply with care — verify, then stand! 🐇📋"

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description does not follow the required template structure. The template specifies required sections (Summary with checkboxes, Acceptance Criteria for MCP UX Hardening with specific AC items, and Validation steps), but the description provides a custom summary and validation without addressing the template's AC items. Update the PR description to follow the required template structure, including Summary section with motivation and linked issues, and Acceptance Criteria as specified in the template, or clarify if a different template applies to this PR type.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add ExStruct CLI skill' directly and clearly describes the main change in the changeset—adding a new installable Skill for ExStruct CLI operations.
Linked Issues check ✅ Passed The PR successfully implements all primary objectives from issue #115: delivers a single installable exstruct-cli Skill with focused SKILL.md, detailed references directory (command-selection, safe-editing, ops-guidance, verify-workflows, backend-constraints), updated README files with installation guidance, documented internal contracts in dev-docs/specs and ADR, and establishes safe workflow patterns for validate/dry-run/apply/verify.
Out of Scope Changes check ✅ Passed All changes are directly aligned with issue #115 requirements. The PR adds the Skill structure, documentation, README updates, ADR/spec files, and task tracking—all within defined scope. No extraneous code, unrelated refactoring, or out-of-scope feature additions are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/cli-skills

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

chatgpt-codex-connector[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

This comment was marked as resolved.

This comment was marked as resolved.

@harumiWeb harumiWeb merged commit 0d99ac7 into main Mar 21, 2026
12 checks passed
@harumiWeb harumiWeb deleted the feat/cli-skills branch March 21, 2026 12:34
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.

Creating ExStruct CLI Skills

2 participants