refactor: target:folder/AGENT.md + rewritten instructions - #1
Merged
Merged
Conversation
Follow the @ctxr/agent-staff-engineer convention: ctxr.target=folder with an AGENT.md entry and no ctxr- prefix in source (the kit adds it to the install dir). Rewrite the agent prompt to a full operating-procedure + output-contract + hard-rules instruction set, replace the boilerplate README with a hand-written one, and drop the stray subagent-dispatch schema (no Tier-2 dispatch here). Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
There was a problem hiding this comment.
Pull request overview
Refactors this agent package to the canonical @ctxr “target: folder” bundle layout, moving the agent definition into AGENT.md and updating the package metadata and README accordingly.
Changes:
- Replace the single-file agent (
ctxr-agent-codebase-explorer.md) with a folder-target bundle entrypoint (AGENT.md) and updated, more explicit operating instructions/output contract. - Update
package.jsontoctxr.target: "folder"and adjust publishedfilesto ship the bundle contents. - Rewrite
README.mdto a shorter, hand-written usage/install overview and remove the unused subagent dispatch schema.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| subagent-dispatch-v1.schema.json | Removed unused schema file from the repo/package surface. |
| README.md | Replaced scaffold README with concise “what/when/install/release” documentation aligned to the new bundle convention. |
| package.json | Switch ctxr.target to folder and publish AGENT.md (and related docs) instead of the old single-file agent. |
| ctxr-agent-codebase-explorer.md | Removed legacy single-file agent definition now superseded by AGENT.md. |
| AGENT.md | New canonical agent entry with rewritten instructions, operating procedure, and strict output contract. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
… on tool errors Remove the tools: allowlist so the agent inherits the full toolset (matching agent-staff-engineer). Make the prompt tool-agnostic (no named tools, MCPs, or connectors, since the tool landscape varies by project and machine) and add a resilience rule: use whatever capabilities exist, treat them as best-effort, and NEVER halt on a missing/unhealthy/erroring tool, note it and fall back. Keep the read-only mission as a firm behavioral rule now that tools no longer enforce it. Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
…privilege claim) The README still claimed the agent carries only built-in tools and no MCP connectors; that contradicts the new full-toolset, never-halt AGENT.md. Reframe to: uses whatever the environment exposes, best-effort, falls back rather than halting, read-only by rule. Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
Resolve the ambiguity between the strict Output contract ("exactly these
sections", no narration) and the resilience rule ("note it in one line"): the
one-line note about a missing/unhealthy capability goes under Open questions.
Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
… note Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
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
ctxr.target: "folder"with anAGENT.mdentry. The kit owns thectxr-prefix (install dirctxr-agent-codebase-explorer); the agent is invoked by its frontmattername: agent-codebase-explorer.tools:restriction, matching agent-staff-engineer), uses whatever capabilities the environment exposes, and never halts on a missing/unhealthy/erroring tool (it notes it and falls back). Read-only is enforced by a firm behavioral rule, not by tool restriction.Test plan
@ctxr/kit validatepasses astarget: folder; markdownlint 0; dash-freetools:restriction; AGENT.md names no concrete tool/MCP/connector (tool-agnostic)npm pack --dry-runships package.json + AGENT.md + README.md + LICENSE🤖 Generated with Claude Code