Conversation
…olidation Update the two remaining references to the old `admin-v1-queries` and `admin-v1-commands` handler names that were not updated when admin/config.yaml was consolidated in commit ab226f8. - module/api_v1_handler.go: Update comment to reference new `admin-queries` and `admin-commands` handler names - docs/PLATFORM_ROADMAP.md: Update inline YAML example to use `admin-queries` The admin/config.yaml itself was already consolidated (16 domain-specific handler modules removed, 2 generic handlers named admin-queries/admin-commands). This commit cleans up the remaining stale references and creates the PR to formally close issue #85. Closes #85 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Updates remaining documentation/comment references to the consolidated admin handler module names (admin-queries, admin-commands) after Phase A admin config consolidation.
Changes:
- Update
V1APIHandlerheader comment to referenceadmin-queries/admin-commands. - Update
docs/PLATFORM_ROADMAP.mdinline YAML example to usehandler: admin-queries.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| module/api_v1_handler.go | Fixes stale handler module name references in the V1APIHandler comment. |
| docs/PLATFORM_ROADMAP.md | Updates the admin route YAML example to use the consolidated handler name. |
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
admin/config.yamlPhase A consolidation (removing 16 domain-specific handler modules, renamingadmin-v1-queries/admin-v1-commandstoadmin-queries/admin-commands) was completed in commit ab226f8Changes:
module/api_v1_handler.go: Update comment to referenceadmin-queriesandadmin-commands(wasadmin-v1-queriesandadmin-v1-commands)docs/PLATFORM_ROADMAP.md: Update inline YAML example to usehandler: admin-queries(washandler: admin-v1-queries)Verification
go build -o /tmp/server ./cmd/server— compiles cleanly./server -config admin/config.yaml— starts successfully,admin-queriesandadmin-commandsmodules register and initializego test ./...— all tests passgolangci-lint run— 0 issuesCloses #85
🤖 Generated with Claude Code