docs: clarify optional agent skills#28
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR primarily addresses Issue #27 by making optional Codex/agent skills discoverable from the root README and clarifying the public vs internal skill boundary for adopting projects. In addition, it expands the TritonKit CLI surface by introducing a shorter triton app go <url> iOS deep-link smoke entry and aligns schemas / capability nextActions / workflow plans / tests with that new command.
Changes:
- Add a new root README section documenting “Optional Agent Skills”, including installation guidance and the public/internal packaging boundary.
- Introduce
triton app go <url>as a shorter iOS deep-link command that waits for embedded runtime readiness and returns a snapshot summary by default. - Update CLI schemas, runtime capability nextActions, workflow plan rendering, and tests/scripts to reference
app goand to accept “defaults to JSON” as machine-readable output.
Reviewed changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| Sources/TritonKitCLI/CLISchemaHostCommands.swift | Adds app go to the command schema/options/examples and updates nextCommands. |
| Sources/TritonKitCLI/CLIRuntimeTransport.swift | Switches capability nextActions / workflow plan command strings to use triton app go. |
| Sources/TritonKitCLI/CLIHostCommands.swift | Implements the new HostAppGo subcommand and iOS-default device selection resolution. |
| README.md | Adds “Optional Agent Skills” section and documents the new app go entry point vs app open-url. |
| docs-linhay/spaces/20260604-issue-27-agent-skills-readme/README.md | Adds an Issue #27 execution space README (background/targets/acceptance). |
| docs-linhay/spaces/20260604-issue-27-agent-skills-readme/plans/implementation.md | Adds an implementation plan for the README skills clarification work. |
| docs-linhay/spaces/20260527-command-surface-optimization/plans/checkpoints/20260602-round-180-short-app-go-deeplink-entry.md | Adds a checkpoint note documenting the app go command rationale and validation. |
| docs-linhay/scripts/verify-simulator-gate.sh | Extends the simulator gate script to run the new URL placeholder canon test. |
| docs-linhay/memory/2026-06-04.md | Adds a memory entry for the Issue #27 README/skills boundary work. |
| docs-linhay/memory/2026-06-02.md | Adds a memory entry describing the app go command addition and related docs/tests. |
| docs-linhay/dev/ai-cli-readable-control.md | Updates agent-facing CLI guidance to prefer app go for iOS deep-link smoke. |
| CLI/Tests/TritonKitCLITests/SchemaFactSourceTests.swift | Updates assertions to expect app go, relaxes machine-readable checks via schema defaults, and adds URL placeholder canon tests. |
| CLI/Tests/TritonKitCLITests/DeviceCrossPlatformTests.swift | Adds schema example expectations for triton app go .... |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -311,7 +314,7 @@ func hostCommandSchemas() -> [TKCommandSchema] { | |||
| outputSemantics: "Use app for host-side install, launch, terminate, open-url, container, and preferences. For smoke readiness, prefer open-url with --wait-ready --snapshot.", | |||
Comment on lines
563
to
+566
| case "host-app-open-url-ready": | ||
| return TKCLINextAction(command: "app", args: ["open-url", "<url>", "--device", "<selector>", "--wait-ready", "--json"]) | ||
| return TKCLINextAction(command: "app", args: ["go", "<url>", "--device", "<selector>"]) | ||
| case "host-app-open-url-snapshot": | ||
| return TKCLINextAction(command: "app", args: ["open-url", "<url>", "--device", "<selector>", "--wait-ready", "--snapshot", "--json"]) | ||
| return TKCLINextAction(command: "app", args: ["go", "<url>", "--device", "<selector>"]) |
Comment on lines
+408
to
+409
| `app go <url>` is the short iOS deep-link smoke entry: it opens the URL, waits for embedded runtime readiness, returns an app/route/AX snapshot summary, and defaults to JSON output. Use `--device <selector>` only when the current/default target is ambiguous. | ||
| For iOS simulator selectors, `sim:` is optional. `--device 60667794-96F8-40E6-8664-85538EC4663E` and `--device sim:60667794-96F8-40E6-8664-85538EC4663E` both resolve to the same simulator; keep `sim:` only when you want explicit platform disambiguation. |
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
Optional Agent Skillssection to the root README..agents/tritonkit-skills/public/ortritonkit-skills.tar.gz.Closes #27
Validation
Result:
docs-linhay structure ok