Add optional ontology check after deep-interview#1
Open
iqdoctor wants to merge 1 commit into
Open
Conversation
iqdoctor
force-pushed
the
scholastic-ralplan-reviewer
branch
from
3489d46 to
b599a7e
Compare
iqdoctor
force-pushed
the
scholastic-ralplan-reviewer
branch
from
b599a7e to
b478a78
Compare
iqdoctor
changed the title
iqdoctor
force-pushed
the
scholastic-ralplan-reviewer
branch
2 times, most recently
from
975219b to
2508a8e
Compare
iqdoctor
force-pushed
the
scholastic-ralplan-reviewer
branch
3 times, most recently
from
9356679 to
8d276fb
Compare
iqdoctor
changed the title
iqdoctor
force-pushed
the
scholastic-ralplan-reviewer
branch
from
8d276fb to
89a91fd
Compare
iqdoctor
force-pushed
the
scholastic-ralplan-reviewer
branch
from
89a91fd to
cb4bbaf
Compare
The previous upstream discussion favored narrow, optional changes over new mandatory planning gates. This keeps the deep-interview handoff semantics intact while exposing a separate ontology-check skill for users who want a Greek-scholastic review before planning or execution. Constraint: Upstream feedback rejected broad readiness vocabulary and mandatory downstream gates. Constraint: Plugin packaging requires mirrored installable skills under plugins/oh-my-codex/skills. Rejected: Inject ontology review into ralplan, ralph, autopilot, plan, or team | changes existing workflow semantics and broadens the PR. Rejected: Add a scholastic agent/prompt | larger surface than needed for an optional skill. Confidence: high Scope-risk: narrow Reversibility: clean Directive: Keep ontology-check advisory unless upstream explicitly accepts a mandatory gate. Tested: npm run build; npm run lint; npm run verify:plugin-bundle; targeted deep-interview/catalog/package tests; catalog check; smoke suite after rebuilding omx-explore-harness. Not-tested: Full npm test after the final mirror-only amend.
iqdoctor
force-pushed
the
scholastic-ralplan-reviewer
branch
from
cb4bbaf to
22498b5
Compare
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
1 participant
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
$ontology-checkskill for Greek-scholastic review of a deep-interview spec or supplied text$deep-interviewrecommend$ontology-check <spec-path>after artifact generation as an advisory pass$ralplan,$autopilot,$ralph,$team, or refine further remain unchangedralplan,autopilot,ralph,team, orplanexecution contracts$ontology-checkkeyword routing, and plugin mirrorUpstream Scope Notes
strato-space/mainis fast-forwarded to upstream0.15.0(0fdbb7ac)mainWhy Optional
The Greek-scholastic pass checks whether the interview result has term drift, ontology failures, logic issues, modality confusion, or weak argument chains. It is useful before planning/execution, but it should not change existing handoff semantics. Users can run it when they want the extra pass; otherwise the normal deep-interview handoff remains intact.
Validation
npm run buildnpm run lintnpm run verify:plugin-bundlenode --test dist/hooks/__tests__/deep-interview-contract.test.jsnode --test dist/catalog/__tests__/schema.test.js dist/catalog/__tests__/generator.test.js dist/cli/__tests__/catalog-contract.test.js dist/cli/__tests__/package-bin-contract.test.jsnode dist/scripts/generate-catalog-docs.js --checkcargo build -p omx-explore-harnessnode --test dist/cli/__tests__/packaged-script-resolution.test.js dist/cli/__tests__/package-bin-contract.test.js dist/cli/__tests__/explore.test.js dist/cli/__tests__/sparkshell-cli.test.js dist/hooks/__tests__/explore-routing.test.js dist/hooks/__tests__/explore-sparkshell-guidance-contract.test.js dist/scripts/__tests__/smoke-packed-install.test.js dist/verification/__tests__/explore-harness-release-workflow.test.js dist/compat/__tests__/*.test.js