Skip to content

feat: adds a skill to scaffold new ECCs - #15

Merged
Ssandor13 merged 1 commit into
mainfrom
add-create-ecc-skill
Jul 16, 2026
Merged

feat: adds a skill to scaffold new ECCs#15
Ssandor13 merged 1 commit into
mainfrom
add-create-ecc-skill

Conversation

@Ssandor13

@Ssandor13 Ssandor13 commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

What this adds

A create-ecc Claude Code skill and a README section documenting it.

What it does today

Given a characteristic to add, the skill runs a research-first workflow — builds
an evidence brief (definition, category, primary citations, how it's
measured/thresholds, NCIt/MONDO alignment), allocates the identifier and
filename, writes a schema-valid entry (respecting state and
deny_unknown_fields), and validates with cargo run -p ecc-cli -- check ecc.

Where it's heading (context, not in this PR)

This is the authoring substrate for a larger goal: mapping characteristics to
CC4K terms as computable logical definitions so classification auto-updates as
knowledge evolves. Full roadmap and open design questions in #16 .

Scope

Skill + README only. No characteristics are shipped in this PR. Adds a local Claude Code skill (.claude/skills/create-ecc/SKILL.md) that
scaffolds a new composable characteristic as a schema-valid YAML file,
mirroring the data model in crates/ecc/src/.

Co-Authored-By: Stephanie Madlom and Claude Opus 4.8 (1M context)

@Ssandor13
Ssandor13 requested a review from claymcleod July 16, 2026 02:04
Comment thread ecc/molec/000002-mycn-amplification.yml Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Are we sure we want to actually adopt this characteristic? Or do you just want the skill in here?

`crates/ecc/src/` — this skill mirrors it. If in doubt, read `crates/ecc/src/lib.rs`,
`crates/ecc/src/common.rs`, `crates/ecc/src/common/value/kind.rs`, and
`crates/ecc/src/identifier.rs` before writing.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would extrapolate this out a bit more for the prior research you want the agent to do. So it should search available publications and present back to you a particular shape the characteristic might take. It should not fabricate anything and should disagree with you when it thinks you've stated something incorrectly.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@claymcleod updated the PR, edited the PR description, and also added an issue #16 .

Adds a local Claude Code skill (`.claude/skills/create-ecc/SKILL.md`) that
scaffolds a new composable characteristic as a schema-valid YAML file,
mirroring the data model in `crates/ecc/src/`. Documents the skill in the
README and adds MYCN amplification as ECC-MOLEC-000002 (binary: amplified vs
not).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Ssandor13
Ssandor13 force-pushed the add-create-ecc-skill branch from 8efeca6 to ddd4f30 Compare July 16, 2026 16:52
@Ssandor13
Ssandor13 requested a review from claymcleod July 16, 2026 19:03

@claymcleod claymcleod left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks pretty good!

@Ssandor13
Ssandor13 merged commit ef222f6 into main Jul 16, 2026
2 checks passed
@Ssandor13
Ssandor13 deleted the add-create-ecc-skill branch July 16, 2026 20:05
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.

2 participants