Add agent authoring entry points#89
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces two new documentation files, docs/AGENT_TOOLKIT.md and docs/llm.txt, to guide agents and LLMs in authoring CraftDAG build plans, and updates docs/LLM_AUTHORING_CONTRACT.md to reference them. The review feedback points out a mismatch in error codes within docs/llm.txt compared to the actual validator implementation, and suggests aligning the CLI option --output to --out in docs/AGENT_TOOLKIT.md for consistency.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| - COMPONENT_OUT_OF_BOUNDS: move anchor inward, reduce size, or increase plan bounds. | ||
| - UNKNOWN_COMPONENT_REF: fix inputs, target, source, or assembly reference. | ||
| - UNKNOWN_MATERIAL_REF: use a palette key or minecraft: block id. | ||
| - BUDGET_EXCEEDED: reduce bounds, repeated counts, levels, sections, or detail components. |
There was a problem hiding this comment.
The error code BUDGET_EXCEEDED does not exist in the validator implementation (packages/core/src/componentPlan.ts). The actual budget-related error codes thrown by the validator are PLAN_BOUNDS_OVER_BUDGET, PLAN_COMPONENTS_OVER_BUDGET, and PLAN_ESTIMATED_BLOCKS_OVER_BUDGET. Updating this to the actual error codes will ensure that LLMs/agents can correctly match and handle these diagnostics during the repair loop.
- PLAN_BOUNDS_OVER_BUDGET / PLAN_COMPONENTS_OVER_BUDGET / PLAN_ESTIMATED_BLOCKS_OVER_BUDGET: reduce bounds, repeated counts, levels, sections, or detail components.
| ```bash | ||
| pnpm --filter @i365dev/craftdag-cli build | ||
| node packages/cli/dist/index.js validate examples/starter-house.craftdag.json | ||
| node packages/cli/dist/index.js compile examples/starter-house.craftdag.json --output /tmp/starter-house.voxel.json |
There was a problem hiding this comment.
There is an inconsistency in the CLI options used for specifying the output file. Here, --output is used, whereas in line 48 (--out /tmp/starter-house.schem) and in the desired CLI contract (lines 67-74), --out is used consistently. Please verify if the current CLI supports --out for the compile command as well, and if so, use it here for consistency.
| node packages/cli/dist/index.js compile examples/starter-house.craftdag.json --output /tmp/starter-house.voxel.json | |
| node packages/cli/dist/index.js compile examples/starter-house.craftdag.json --out /tmp/starter-house.voxel.json |
madawei2699
force-pushed
the
codex/agent-authoring-pack
branch
from
3992fa0 to
039107d
Compare
1 participant
Summary
Refs #54
Refs #55
Validation