Repository navigation
feat(core): name recipes .ocw-in.json and .ocw-out.json - #256
Merged
Merged
Conversation
… them beside other JSON A folder holding any .ocw-in.json or .ocw-out.json file is read for those plus any other file whose kind is input or output, so package.json and data files no longer fail the load. A folder without them reads as before, and a file named for one kind that holds the other is an error. Studio's New recipe and document import write the new names, and the VS Code extension validates both against their schemas. Co-Authored-By: Claude Sonnet 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_014yC6LN4a8qnR7gu3QmRnbg
Co-Authored-By: Claude Sonnet 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_014yC6LN4a8qnR7gu3QmRnbg
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.
Why
A recipe was any
.jsonin a folder, so one among other JSON files was easy to lose, and a straypackage.jsonfailed the whole load.What
<id>.ocw-in.jsonis an input recipe,<id>.ocw-out.jsonan output recipe. Still plain JSON.readRecipeFiles, used by the CLI, MCP and Studio): with any marked file present, those are read plus any other.json/.jsonlwhosekindisinputoroutput; other JSON beside them is left alone. A folder with none reads as before, so existing*.input.json/*.output.jsonkeep working.kindinside the file stays the authority.jsonValidationmaps*.ocw-in.jsonand*.ocw-out.jsonto the input and output schemas (needs an extension version bump to ship; see feat(vscode): release 0.1.0 #255).docs/recipes/authoring.md.assets/(logos and icons) as its own commit.Not in this PR
Renaming the 196 tracked
*.input.json/*.output.jsonfiles (examples, test fixtures, docs) to the new names. They keep working; that is a mechanical follow-up.Test plan
nx run-many -t lint,typecheck,test,build -p core cli studio studio-ui mcp vscode🤖 Generated with Claude Code