fix: preserve safety sections in Skill distillation - #617
Open
kriptoburak wants to merge 1 commit into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Preserve request workflows and safety controls during Skill-to-GEP parsing.
The defect surfaced against the public Xquik X Twitter Scraper Skill.
What changed
Before this change, the sample shape produced only workflow examples. It
dropped the ordered request process and every safety section. After this
change, the same shape keeps both the process and its guardrails.
How to test
node --test test/skill2gepParser.test.js.node --test test/skill2gep.test.js test/skill2gepParser.test.js test/skill2recipes.test.js.npm test.node index.js --help.Expected results:
Risk
Medium. New Genes may contain more source Skill rules. Existing assets,
schemas, network behavior, and the 28-step cap remain unchanged.
Harness/evaluator governance
Upstream governance surface:
src/gep/skill2gep.jsheading selection for reverse-distilled Gene strategies.Downstream EvoX impact: New reverse distillations preserve additional source rules; schemas and bridge contracts stay unchanged.
Rollout-local scope: The change affects newly parsed local Skills only; existing assets remain unchanged.
Promotion boundary: Merge enables parsing; existing review, publication, and promotion gates remain separate.
Evaluator mismatch sets: Observation recognizes more headings; action retains their list items; repair adds aliases; verification covers parser and full-suite behavior; evidence includes a live public Skill probe; belief logic is unchanged.
Non-regression evidence: 24 parser tests, 52 focused Skill tests, 3,290 full-suite tests, CLI sanity, and a live public Xquik Skill parse.
Fix-severity review: medium
Owner approval: Skill-to-GEP parser owner review required.
Security boundary: No new data, tool, host, network, or secret access. Credential and untrusted-content rules are retained instead of dropped.
Rollback: Revert this commit. Existing assets require no migration.
Live promotion: no
Autonomous evaluator self-editing: no
Self-check
Related
No existing issue or pull request covers this parser gap.