Repository navigation
feat(minigraph-playground): deploy tutorials 2 to 14 as a graph set; drop the import advisory's redundant sentence - #567
Merged
Conversation
…drop the import advisory's redundant sentence
The Playground example is now the worked example of graph sets (ADR-0027): tutorial-1 and
support-triage stay loose graphs, and tutorials 2 to 14 deploy as the set 'tutorials'
(resources/graph/tutorials.pack, version 1.0) unpacked into file:/tmp/unpack. The pack is
the graph packager's output for the engine's tutorial JSON files, byte for byte.
The import fallback no longer prints "Please export an updated version and re-import to
instantiate an instance model" after "Found deployed graph model in {location}" - the
following "Graph model imported as draft" already says what happened. The five tutorial
help pages that quote the output follow, and the Playground bundle is rebuilt
(index-BTVKYfNr).
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ort advisory Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…efuses a folder inside the Playground's temp folder
The help page 'help package', the starter template's commented manifest and the AI agent
guide's deploy-a-set steps showed an unpack folder under /tmp/graph, which the graph-set
loader refuses at startup ("must not be the Playground's temporary folder
(location.graph.temp) or inside it"). They now use file:/tmp/unpack and state the rule, and
the configuration reference's refusal sentence names a folder inside the temp folder too.
GraphSetLoaderTest keeps /tmp/graph/unpacked as its negative case. The Playground bundle is
rebuilt for the help page (index-BVYn3P8C).
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
What
examples/minigraph-playgroundkeepstutorial-1andsupport-triageas loose graphs and deploys tutorials 2 to 14 as the settutorials(resources/graph/tutorials.pack, version 1.0), unpacked intofile:/tmp/unpack. The pack is the graph packager's output for the engine'stutorial-N.jsonfiles, byte for byte (pack --set tutorials --manifest 'description=Tutorial-2 to 14' --manifest version=1.0, SHA-25633107f91…), soimport graph from tutorial-Nandlist graphsnow show the set at work (tutorial-6 - … (set tutorials, version 1.0)).Found deployed graph model in {location}the console no longer prints "Please export an updated version and re-import to instantiate an instance model" - the following "Graph model imported as draft" already says what happened. The five tutorial help pages that quote the output (5, 6, 8, 10, 12) follow, and the Playground bundle is rebuilt (index-CLmQ6fcx→index-BTVKYfNr; the chunk's diff is those five help lines).unpackfolder inside the Playground's temp folder (location.graph.temp), yethelp package, the starter template's commented manifest and the AI agent guide's deploy-a-set steps showed one under/tmp/graph- refused at startup. They now usefile:/tmp/unpackand state the rule; the configuration reference's refusal sentence adds "or a folder inside it".GraphSetLoaderTestkeeps/tmp/graph/unpackedas its negative case. Bundle rebuilt again (index-BVYn3P8C).Why
Graph sets had a packager, a loader, a panel and a configuration reference, but no running example: a reader had to assemble the manifest from the reference. The Playground example now deploys loose graphs and a set side by side, and its tests exercise the set (GraphTests' tutorials and the tutorial-14 suspend/resume drive reach their graphs through it). The advisory line asked the user to do something the import had just done for them.
Lock-step twin in the Rust port: Accenture/mercury#387, with the identical pack.
Verification
mvn -o install -f system/minigraph-playground-engine/pom.xml- 227 tests, 0 failuresmvn -o test -f system/ai-contract-provider/pom.xml- 20 tests, 0 failures (the snapshot's link check)mvn -o test -f examples/minigraph-playground/pom.xml- 23 tests, 0 failures; the log readsDeployed set tutorials (version 1.0) from classpath:/graph/tutorials.pack - 13 graphs into file:/tmp/unpackNote for later: the tutorials now live twice in this repo - the engine's loose JSON (the source) and the example's pack - so a tutorial edit needs a re-pack with the command above.
Co-Authored-By: Claude Opus 5.5 noreply@anthropic.com
🤖 Generated with Claude Code