Repository navigation
fix: v1.0.6 — the last Claude Directory changes, listing paused - #28
Merged
Merged
Conversation
The Directory's blocking "." stayed after v1.0.5, still named by triage-first.sh. Its reminder is the one line, unchanged since v1.0.1, with an escaped quote and a ">" before a period: \"Fast lane — …; model: <sonnet|opus>.\" — a parser that doesn't take \" as a quote ends the string there and reads `opus>.` as a redirect to ".". The reminder now quotes the triage line with single quotes, same text. check-env-declared builds its patterns from quote variables (the 15 expand byte for byte as before), _lib.sh unquotes settings without escapes, and session-context's detached-HEAD line no longer ends "<slug>.". The static check refuses escaped quotes and ">." in the plugin's hooks; a negative control flags them on main. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The branch scan showed the reminder fix worked — the blocking "." no longer follows triage-first.sh — but a new "." followed check-env-declared.sh and the _lib.sh it loads, both clean in v1.0.5 and rewritten in the previous commit without evidence. Both go back to v1.0.5 exactly; v1.0.6 is the two message lines alone. The static check drops the escaped-quote rule and keeps the ">." rule. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The branch scan dropped the "." from the list of files the hooks name,
but the blocking finding still names ".". /aplyca-adf:adopt and
/aplyca-adf:upgrade named ${CLAUDE_PLUGIN_ROOT}/../.. — from
plugins/aplyca-adf, the repository root, outside the plugin's tree,
which the validator reports as "." — to find a development checkout.
The marketplace's installLocation already is that checkout when the
marketplace was added from a local folder, so the path goes. The static
check refuses any path that climbs out of the plugin.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The branch scans showed the reminder fix and the out-of-plugin path each removed something the Directory's validator pointed at, but the blocking finding remains, its source unknown. The listing is paused. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
mauricios
force-pushed
the
fix/directory-escaped-quotes
branch
from
October 5, 2026 02:01
3f620a6 to
0f33931
Compare
mauricios
changed the title
mauricios
marked this pull request as ready for review
October 5, 2026 02:03
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
1 participant
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.
What changed and why
v1.0.6, a patch release, and the last of the Claude Directory work for now. The listing is paused. Installing from GitHub, per project, is unaffected.
triage-first.sh's reminder quotes the one-line triage with single quotes, with no\"and no period after a>. The text is otherwise the same. A branch scan showed this removed the.the validator listed aftertriage-first.sh.session-context.sh's detached-HEAD line no longer ends in<slug>./aplyca-adf:adoptand/aplyca-adf:upgradeno longer name${CLAUDE_PLUGIN_ROOT}/../.., a path outside the plugin. A development install finds the framework through the marketplace'sinstallLocation, which is the local checkout when the marketplace was added from one.>before a period in the plugin's hooks, and any path that climbs out of the plugin.An earlier commit on this branch rewrote
check-env-declared.shand_lib.sh. A scan showed that introduced a new., so both are restored to v1.0.5 exactly; the net diff doesn't touch them.Still unresolved: the validator's blocking
COMMAND_PATH_COMPUTEDat., with no file or line. Its source wasn't found. Bisecting it on a scratch branch is the way to find it when the listing resumes.Upgrade impact
Overwrite:
session-context.shandtriage-first.sh(message text only).How to verify
./evals/run-evals.sh.Verified / not verified
8393539and1e63367.Merge danger
🤖 Generated with Claude Code