Remove reload command from all configs and docs#59
Conversation
- Delete reload.md command definition from packages/core/commands/ - Remove reload.md from packages/opencode/.opencode/commands/ - Delete reload.mdx documentation from packages/web/src/content/docs/docs/reference/commands/ - Update kompass.jsonc files to remove reload command references - Update CommandShowcase.astro and commands index.mdx to remove reload references
There was a problem hiding this comment.
★★☆☆☆
This PR removes the reload command from configs and docs, but misses several critical files that still reference the reload command:
Critical Issues (will break build/tests):
- kompass.schema.json - The JSON schema still references
reloadas a valid command in 3 locations:
- Line 62-64:
"reload": { "$ref": "#/$defs/commandConfig" } - Line 106: In the
enabledarray enum for valid commands - Line 134: In the
templatesproperty names enum
These must be removed to keep schema validation in sync. Leaving them will cause schema validation failures.
- packages/opencode/test/commands-config.test.ts - Multiple tests expect the reload command to exist:
- Line 37:
"reload"appears inexpectedCommandsarray - Line 64:
assert.equal(cfg.command!["reload"]?.agent, "build"); - Line 75:
assert.ok(cfg.command!["reload"]?.template); - Lines 115-125: Test that verifies reload template rewriting
These tests will fail after the command is removed.
Medium Issues:
- packages/web/src/content/docs/docs/reference/tools/index.mdx - Line 59 states
- used by: /reloadwhich references the removed command. This documentation should be updated since the reload tool still exists internally but no longer has a user-facing command wrapper.
What was done correctly:
- All configuration files correctly removed
reloadfrom thecommandssection - The
reloadtool remains in thetoolssection (correct, as this is a low-level tool) - All command documentation files were deleted
- Command showcase and index were updated
Required fixes before merge:
- Remove
reloadcommand references from kompass.schema.json - Update tests in packages/opencode/test/commands-config.test.ts to remove reload expectations
- Update packages/web/src/content/docs/docs/reference/tools/index.mdx line 59
Run bun run compile && bun run typecheck && bun run test after making these fixes to verify nothing breaks.
|
All review feedback has been addressed: Fixed:
Validation:
No additional steps are required. |
There was a problem hiding this comment.
★★★☆☆
The previous review feedback has been properly addressed - schema, tests, and docs are updated correctly. However, one critical source of truth was missed.
Missing: command definition in packages/core/commands/index.ts
The reload command definition (lines 45-50) still exists in packages/core/commands/index.ts, but the corresponding template file packages/core/commands/reload.md has been deleted. According to AGENTS.md, these files are part of the linked surface area that must stay in sync.
Fix required: Remove the reload command definition from packages/core/commands/index.ts (lines 45-50):
- reload: {
- description: "Reload the current OpenCode project configuration",
- agent: "build",
- templatePath: "commands/reload.md",
- subtask: false,
- },After removing this, re-run bun run compile && bun run typecheck && bun run test to verify the removal is complete.
What was done correctly:
- ✅ All config files (root, core, opencode) removed reload from commands
- ✅ Schema removed reload command references
- ✅ Tests updated to remove reload expectations
- ✅ Web docs updated for reload tool
- ✅ Command docs showcase and index updated
- ✅ Tool documentation (reload.mdx) correctly retained since the tool still exists
Ticket
SKIPPED
Description
Remove the reload command from all configuration files and delete associated documentation files. The reload functionality is superseded by automatic project cache management.
Checklist
Configuration Updates
Documentation Cleanup
Integration Updates
Validation