@astryxdesign/cli 0.7.2-canary.fdd384f → 0.7.2

package/clients/cli/commands/integration.mjs

@@ -20,6 +20,7 @@ import {doc as integrationAddFn} from '../../../api/integration/integrationAdd.d
 import {doc as integrationCheckFn} from '../../../api/integration/integrationCheck.doc.mjs';
 import {printIntegrationCheckResponse} from './doctor.mjs';
 import {doc as integrationPackCheckFn} from '../../../api/integration/integrationPackCheck.doc.mjs';
+import {formatCliCommand} from '../../../foundation/env/package-manager.mjs';
 
 /**
  * @param {import('commander').Command} command
@@ -79,10 +80,16 @@ async function verifyPackage(program) {
     if (issues.length > 0) {
       blocks.push(
         list(
-          issues.map(
-            issue =>
-              `[${issue.severity === 'error' ? 'fail' : 'warn'}] ${issue.message}`,
-          ),
+          issues.map(issue => [
+            `[${issue.severity === 'error' ? 'fail' : 'warn'}] ${issue.message}`,
+            // Every field the JSON issue carries.
+            ...(issue.file != null
+              ? [
+                  `file: ${issue.file}${issue.line != null ? `:${issue.line}` : ''}`,
+                ]
+              : []),
+            ...(issue.fix != null ? [`fix: ${issue.fix}`] : []),
+          ]),
         ),
       );
     }
@@ -155,7 +162,8 @@ async function checkPackage(program, options) {
         : data.name;
     emit(
       // --strict lists each warning and each skipped check again, as the
-      // error it now is, in the issue format the checks use.
+      // error it now is: by check, code and place only, since the check's own
+      // output above already printed its message and fix once.
       ...(data.failures.length > 0
         ? [
             section('Failing under --strict'),
@@ -166,8 +174,6 @@ async function checkPackage(program, options) {
                 'code',
                 'component',
                 'prop',
-                'message',
-                'fix',
                 'file',
                 'line',
               ],
@@ -278,14 +284,14 @@ export function registerIntegration(program) {
       if (!options.check) {
         // One way forward: the new name. `npm pack` is what builds a tarball.
         return cliError(
-          '`integration pack` is now `integration verify`: run `astryx integration verify` to check the package the way npm will publish it. To build the tarball, run `npm pack`.',
+          `\`integration pack\` is now \`integration verify\`: run \`${formatCliCommand('integration verify')}\` to check the package the way npm will publish it. To build the tarball, run \`npm pack\`.`,
           {code: ERROR_CODES.ERR_INVALID_ARGUMENT},
         );
       }
       // stderr, and only in text: JSON callers get the same envelope as verify.
       if (!program.opts().json) {
         console.error(
-          'Note: `integration pack --check` is deprecated. Run `astryx integration verify`: it runs the same check.',
+          `Note: \`integration pack --check\` is deprecated. Run \`${formatCliCommand('integration verify')}\`: it runs the same check.`,
         );
       }
       return verifyPackage(program);

package/clients/cli/commands/json-contract.test.mjs

@@ -143,7 +143,12 @@ describe('--json contract: supported commands emit valid envelopes', () => {
     const parsed = parseJson(stdout);
     expect(parsed.type).toBe('discover.list');
     expect(parsed.data).toEqual([]);
-    expect(parsed.meta).toEqual({configured: false});
+    expect(parsed.meta.configured).toBe(false);
+    // The official list the CLI carries is always a source.
+    expect(parsed.meta.sources).toEqual([
+      expect.objectContaining({name: 'Astryx', status: 'fresh'}),
+    ]);
+    expect(parsed.meta.answeredBy).toEqual(expect.any(Object));
   });
 
   it('astryx template --json emits a typed envelope', async () => {
@@ -162,7 +167,11 @@ describe('--json contract: supported commands emit valid envelopes', () => {
   });
 
   it('astryx doctor --json emits a doctor envelope with checks + summary', async () => {
-    // Run in a bare tmp dir → @astryxdesign/core won't resolve → a FAIL → exit 1.
+    // A project that declares @astryxdesign/core but installs none → a FAIL → exit 1.
+    fs.writeFileSync(
+      path.join(tmpDir, 'package.json'),
+      '{"name":"app","dependencies":{"@astryxdesign/core":"*"}}',
+    );
     const {status, stdout} = await runCli(['doctor', '--json'], {cwd: tmpDir});
     expect(status).toBe(1);
     const parsed = parseJson(stdout);

package/clients/cli/commands/manifest.doc.mjs

@@ -22,6 +22,7 @@ export const doc = {
     {label: 'Full manifest', cli: 'astryx manifest --json'},
     {label: 'The same manifest, under data.manifest (type "help")', cli: 'astryx --json'},
   ],
+  outsideProject: 'same',
   exitCodes: [
     {code: 0, when: 'success'},
     {code: 1, when: 'the manifest cannot be generated'},