Repository navigation
fix: make adoption and diagnostics safe for linked project setups (#533) - #538
Merged
Merged
Conversation
- Normalize raw consumer SDLC domain configuration in maintenance doctor without mutating policy files (AC1) - Recognize linked canonical skills and prevent skill adapter sync from writing through symlinks (AC2) - Preview, apply, verify, and roll back generated harness adapters transactionally during adoption (AC3) - Seed declared, locked project-scoped Ajv/yaml dependencies for Rust consumers and load OpenTelemetry lazily when telemetry is disabled (AC4) - Distinguish unsupported automatic Rust test/CI discovery from proven absence in init and posture checks (AC5) Closes #533 Refs: #516 Role-Pass: developer
5 tasks
This was referenced Oct 18, 2026
Closed
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.
Make adoption, adapter synchronization, and maintenance diagnostics safe for linked consumer setups and Rust repositories without Node manifests.
inspectDoctor(lib/services/doctor-readers.mjs) so raw consumersdlc.config.jsonand effective config receive consistent structural diagnostics without rewriting project policy (AC1).lib/skill-adapters.mjsandlib/services/doctor-readers.mjsso canonical linked skills are recognized aslinked-canonical(non-stale), non-canonical/broken links are reported accurately, andwrite: truenever writes through symlinks into canonical source (AC2)..agentflow/roles/**,.claude/commands/**,.pi/commands/**, plugin manifests, structural-merged settings, and existing non-linked skill adapters) transactionally with recoverable preimages (mode: 'sync'), and surface stale generated adapters inonboarding verifyand readiness reports (AC3).manifests/composition-profiles.jsonandmanifests/product-payload.jsonin the managed payload, load@opentelemetry/*lazily when telemetry is disabled, and seed declared/lockedajv@8.20.0andyaml@2.9.1dependencies for Rust (Cargo.toml) consumers (AC4).status: 'unsupported', not proven absent) from proven absence indetectTestCommand,assumptionNotes, andcheckPostureCapability(AC5).Implemented issues
Closes #533
Related issues
#516; Dendro smota/dendro#62
CI-equivalent validation
Status: passed
Commands:
pnpm validate:release;inspectDoctorandrunConfigDoctoragainst/home/sam/code/dendro.Results: all test suites, hooks, workflow, distribution, authority, extension, skill, role, path, docs, reachability, evals, collaboration, package-parity, sandbox, cockpit, and npm-package checks passed.
Workflow evidence
Workflow-status comment: #533 (comment)
Handover comments: exception: single-session orchestration with bounded subagent delegation
Role-pass summary: Agy orchestrated architecture and integration; Grok (
xai/grok-4.7-build-fast) implemented Slice A and Slice B; Grok (xai/grok-4.7) executed independent cross-family QA and diff review (APPROVE).Validation evidence: full
pnpm validate:releaseand live read-only consumer verification recorded in the workflow-status comment.Agent review
Implemented by: agy
Launcher: antigravity
Executor: agy-session
Transport: local-cli
Delegation boundary: current-session
Model / runtime: Gemini 4 Argon (low) orchestrator; delegated builder/reviewer via pi-cli (
xai/grok-4.7-build-fastandxai/grok-4.7, low)Review: self-review
Workflow profile: standard
Merge owner: human/operator
Mode: single-agent
Self-review disclosure: Agy performed final synthesis and self-review; cross-family review was also performed by
xai/grok-4.7viapi -p(APPROVE).Regression test: added
Follow-up issues
None created; #534 and #535 remain tracked in the backlog.