Rename the Reviews skill to Inspect - #12
Merged
Merged
Conversation
ArchDev v0.38.0 renamed the `reviews` command tree to `inspect` and keeps `reviews` as a deprecated alias. The skill follows: `skills/inspect`, every command spelling, the bootstrap probes (`inspect local`, `inspect workflows run`, `inspect manifest`), the packaging test, the workflow job, and the README install section. No `reviews` stub is kept; installed copies keep working through the CLI alias.
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.
Review on ArchCode
Problem and author intent
ArchDev
v0.38.0(firstlanding #13380) renamed thereviewscommand tree toinspect, keepingreviewsas a deprecated alias that prints a warning. The landing page installs the product as--skill inspect. This repo's skill must carry the same name and command spellings before that page goes to production.What changed
skills/reviewsmoved toskills/inspect;name: inspect; every command inSKILL.mdand the three references now readsarchdev inspect …..shand.ps1) probesinspect local --help,inspect workflows run --help, andinspect manifest --help, so any CLI older than0.38.0is upgraded.tests/inspect-skill.shand theInspect Skill Installationworkflow job replace the reviews ones. README section renamed with--skill inspect.reviewsstub skill is kept. Copies already installed keep working: the CLI alias runs the same commands with one warning line.Not renamed on purpose:
.archdev/reviews/*.json, the review-workflow definition directory, and prose uses of "reviews" as a verb or as GitHub reviews.Testing
bash tests/inspect-skill.shpasses: bothnpx skills addscopes, then cold, reused, outdated, and failed-installer bootstrap against a fixture CLI.archdevon PATH installed0.38.0from the official installer and passed all threeinspectprobes;archdev reviews guideon that binary prints the deprecation warning and still works.Follow-up
Promote the ArchDev landing page to production only after this merges; it copies
npx skills add ArchAstro/archdev --skill inspectverbatim.🤖 Generated with Claude Code
https://claude.ai/code/session_01SeihH5jFmVuB1GsaNkFhpA