Skip to content

feat(cli): add integration authoring checks to doctor - #6173

Merged
josephfarina merged 1 commit into
mainfrom
doctor-integration-authoring
Sep 9, 2026
Merged

feat(cli): add integration authoring checks to doctor#6173
josephfarina merged 1 commit into
mainfrom
doctor-integration-authoring

Conversation

@josephfarina

@josephfarina josephfarina commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • make doctor integration validate the only structural validation command and remove the old top-level command
  • add doctor integration templates and doctor integration components to warn about Core identity conflicts and print the exact package-qualified command
  • add doctor integration docs to distinguish intentional replaces/extends relationships from accidental same-name Core conflicts
  • preserve the command result-reporting contract added by feat(cli): make every command report what it returned #6172

Test plan

  • pnpm -F @astryxdesign/cli exec tsc --project tsconfig.api-dts.json
  • pnpm -F @astryxdesign/cli typecheck:authoring
  • pnpm -F @astryxdesign/cli typecheck:json-api
  • pnpm lint:strict
  • pnpm check:repo
  • pnpm exec vitest run packages/cli (3,572 passed)
  • pnpm build
  • exercised the real CLI command tree and confirmed validate-integration is unknown

@vercel

vercel Bot commented Sep 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
astryx Ready Ready Preview Sep 9, 2026 5:52pm UTC

Request Review

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Meta Open Source bot. label Sep 9, 2026
@josephfarina
josephfarina force-pushed the doctor-integration-authoring branch from cfafb99 to 4bcc8fb Compare September 9, 2026 02:28
github-actions Bot added a commit that referenced this pull request Sep 9, 2026
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

PR Analysis Report

📚 Storybook Preview

View Storybook for this PR
GitHub Pages may take up to a minute to hydrate after deploy.

🧪 Sandbox Preview

View Sandbox for this PR
GitHub Pages may take up to a minute to hydrate after deploy.

No new or modified components detected.

Bundle Size Summary

No component packages changed.

Accessibility Audit

Status: No accessibility violations detected.


Generated by PR Enrichment workflow | Storybook | Sandbox | View full report

@josephfarina
josephfarina force-pushed the doctor-integration-authoring branch from 4bcc8fb to ed7be09 Compare September 9, 2026 04:28
@josephfarina josephfarina changed the title feat(cli): group integration checks under doctor feat(cli): add integration authoring checks to doctor Sep 9, 2026
@josephfarina
josephfarina marked this pull request as ready for review September 9, 2026 17:35
@josephfarina
josephfarina force-pushed the doctor-integration-authoring branch from ed7be09 to 419b0b0 Compare September 9, 2026 17:46
github-actions Bot added a commit that referenced this pull request Sep 9, 2026
@josephfarina
josephfarina merged commit 54ecd59 into main Sep 9, 2026
27 checks passed
@github-actions
github-actions Bot deleted the doctor-integration-authoring branch September 10, 2026 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant