You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Teach artifact findings and project knowledge in the ArchDev skill - #80
Teach capability-gated archdev log knowledge and artifact-backed archdev log finding in the canonical skill and monitor reference.
Cover bounded lexical recall, omitted history/content, attribution and local verification; preview then deliberate --publish; evidence/applicability; and linked updates or corrections without erasing history.
Preserve the top-level-only sharing rule, optional no-finding outcomes, authorization/secrets checks, and the distinction between session-selected projects and a shared checkout.
Check advertised commands and flags independently. Older CLIs retain reviewed log search / log post fallbacks without forced upgrades, hook changes, or bypassing a supported command's rejected lookup.
Regenerate both documents in all eight plugin bundles and add two generator regression tests. Correct blanket delivery/project-validation statements that do not apply to the new read-only or preview paths.
Verification
npm run typecheck
npm test: 127 passed, including both new documentation/distribution regressions.
npm run check: 148 generated files match.
Canonical skill/reference match all eight generated copies byte-for-byte; skill descriptions remain within 1,024 characters.
git diff --check
Read-only help checks: the installed 0.54.6 CLI does not advertise these commands; the companion implementation build exposes the documented commands and flags.
No installed skills/hooks, plugin wiring, catalog/package versions, or unrelated risk/rules guidance changed.
Documentation/tests do not establish live agent uptake or verified reuse. Real-harness evaluation remains follow-up work; conversation delivery into coding sessions is separate.
Reviewed the published assessments for the exact PR head c894e8c76294592d478e254d59fe725e482fa7b7: PR risk is Medium; the six available code-region assessments are Low. The concerns are about CLI-contract evidence and mirrored guidance, not a demonstrated contradiction.
Verification behind the guidance:
The two added generator regressions check advertised-command/flag gating, safe fallback boundaries, preview versus explicit publication, required evidence/project fields, public message IDs, omission/uncertainty language, and exact distribution to all eight harnesses.
All 127 generator tests pass; all 20 GitHub CI checks pass on this head. The generator's existing byte-for-byte distribution and description-length tests also pass.
Read-only help checks confirm the installed 0.54.6 CLI lacks finding/knowledge, while the local companion implementation advertises all documented flags. Its command tests check that previews perform reads only, publication uses the normal lesson/delivery path, rejected projects/linked messages do not publish, and recalled posts remain untrusted information.
The companion implementation is now available for independent review in firstlanding#19406, head 18fbbddfda0be3940415e2acce7b27596a9c16b5: command definitions and preview/publication checks, command and session-routing regressions, and hook integration. Its refreshed focused verification has 222 passing tests. The author approved a one-commit local Aster bypass for the documented baseline-only sandbox failures; normal hosted CI remains the gate for that implementation. These checks do not establish live agent uptake or verified reuse; that remains follow-up work. No change was made solely to lower the server grade.
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
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
archdev log knowledgeand artifact-backedarchdev log findingin the canonical skill and monitor reference.--publish; evidence/applicability; and linked updates or corrections without erasing history.log search/log postfallbacks without forced upgrades, hook changes, or bypassing a supported command's rejected lookup.Verification
npm run typechecknpm test: 127 passed, including both new documentation/distribution regressions.npm run check: 148 generated files match.git diff --checkRollout and scope