@vltpkg/cli-sdk 1.0.7 → 1.0.8

package/dist/commands/bugs.js

@@ -1,10 +1,10 @@
 import { error } from '@vltpkg/error-cause';
 import { PackageInfoClient } from '@vltpkg/package-info';
 import { Spec } from '@vltpkg/spec';
-import { urlOpen } from '@vltpkg/url-open';
 import { actual } from '@vltpkg/graph';
 import { Query } from '@vltpkg/query';
 import { SecurityArchive } from '@vltpkg/security-archive';
+import { openUrl } from "../open-url.js";
 import { createHostContextsMap } from "../query-host-contexts.js";
 import { commandUsage } from "../config/usage.js";
 import hostedGitInfo from 'hosted-git-info';
@@ -138,7 +138,7 @@ export const command = async (conf) => {
             if (!result) {
                 throw error('Unexpected empty result');
             }
-            await urlOpen(result.url);
+            await openUrl(result.url);
             return result;
         }
         // Multiple results, return the list
@@ -158,6 +158,6 @@ export const command = async (conf) => {
     }
     /* c8 ignore stop */
     // Open the URL
-    await urlOpen(url);
+    await openUrl(url);
     return { url, name };
 };

package/dist/commands/docs.js

@@ -1,10 +1,10 @@
 import { error } from '@vltpkg/error-cause';
 import { PackageInfoClient } from '@vltpkg/package-info';
 import { Spec } from '@vltpkg/spec';
-import { urlOpen } from '@vltpkg/url-open';
 import { actual } from '@vltpkg/graph';
 import { Query } from '@vltpkg/query';
 import { SecurityArchive } from '@vltpkg/security-archive';
+import { openUrl } from "../open-url.js";
 import { createHostContextsMap } from "../query-host-contexts.js";
 import { commandUsage } from "../config/usage.js";
 import hostedGitInfo from 'hosted-git-info';
@@ -128,7 +128,7 @@ export const command = async (conf) => {
             if (!result) {
                 throw error('Unexpected empty result');
             }
-            await urlOpen(result.url);
+            await openUrl(result.url);
             return result;
         }
         // Multiple results, return the list
@@ -148,6 +148,6 @@ export const command = async (conf) => {
     }
     /* c8 ignore stop */
     // Open the URL
-    await urlOpen(url);
+    await openUrl(url);
     return { url, name };
 };

package/dist/commands/list.js

@@ -4,6 +4,7 @@ import { SecurityArchive } from '@vltpkg/security-archive';
 import { error } from '@vltpkg/error-cause';
 import { commandUsage } from "../config/usage.js";
 import { createHostContextsMap } from "../query-host-contexts.js";
+import { createGetAuthHeader } from "../query-auth.js";
 import { MermaidImageView } from "../mermaid-image-view.js";
 export const needsRegistry = true;
 export const usage = () => commandUsage({
@@ -104,6 +105,7 @@ export const command = async (conf) => {
     const projectQueryString = ':workspace, :project > *';
     const selectImporters = [];
     const hostContexts = await createHostContextsMap(conf);
+    const getAuthHeader = createGetAuthHeader(conf);
     const importers = new Set();
     const scopeIDs = [];
     // handle --scope option to add scope nodes as importers
@@ -123,6 +125,7 @@ export const command = async (conf) => {
             importers,
             securityArchive,
             hostContexts,
+            getAuthHeader,
         });
         const { nodes: resultNodes } = await scopeQuery.search(scopeQueryString, {
             signal: new AbortController().signal,
@@ -174,6 +177,7 @@ export const command = async (conf) => {
         importers: importers_,
         securityArchive,
         hostContexts,
+        getAuthHeader,
     });
     const query = queryString ||
         /* c8 ignore next */

package/dist/commands/query.js

@@ -6,6 +6,7 @@ import { createDiffFilesProvider } from "../query-diff-files.js";
 import { SecurityArchive } from '@vltpkg/security-archive';
 import { commandUsage } from "../config/usage.js";
 import { createHostContextsMap } from "../query-host-contexts.js";
+import { createGetAuthHeader } from "../query-auth.js";
 import { MermaidImageView } from "../mermaid-image-view.js";
 export const needsRegistry = true;
 export const usage = () => commandUsage({
@@ -130,6 +131,7 @@ export const command = async (conf) => {
     const queryString = targetQueryString || positionalQueryString;
     const hostContexts = await createHostContextsMap(conf);
     const diffFiles = createDiffFilesProvider(conf.options.projectRoot);
+    const getAuthHeader = createGetAuthHeader(conf);
     const importers = new Set();
     const scopeIDs = [];
     // Handle --scope option to add scope nodes as importers
@@ -150,6 +152,7 @@ export const command = async (conf) => {
             securityArchive,
             hostContexts,
             diffFiles,
+            getAuthHeader,
         });
         const { nodes: resultNodes } = await scopeQuery.search(scopeQueryString, {
             signal: new AbortController().signal,
@@ -214,6 +217,7 @@ export const command = async (conf) => {
         securityArchive,
         hostContexts,
         diffFiles,
+        getAuthHeader,
     });
     const query = queryString ||
         (graph ? defaultProjectQueryString : defaultLocalScopeQueryString);

package/dist/commands/repo.js

@@ -1,10 +1,10 @@
 import { error } from '@vltpkg/error-cause';
 import { PackageInfoClient } from '@vltpkg/package-info';
 import { Spec } from '@vltpkg/spec';
-import { urlOpen } from '@vltpkg/url-open';
 import { actual } from '@vltpkg/graph';
 import { Query } from '@vltpkg/query';
 import { SecurityArchive } from '@vltpkg/security-archive';
+import { openUrl } from "../open-url.js";
 import { createHostContextsMap } from "../query-host-contexts.js";
 import { commandUsage } from "../config/usage.js";
 import hostedGitInfo from 'hosted-git-info';
@@ -132,7 +132,7 @@ export const command = async (conf) => {
             if (!result) {
                 throw error('Unexpected empty result');
             }
-            await urlOpen(result.url);
+            await openUrl(result.url);
             return result;
         }
         // Multiple results, return the list
@@ -152,6 +152,6 @@ export const command = async (conf) => {
     }
     /* c8 ignore stop */
     // Open the URL
-    await urlOpen(url);
+    await openUrl(url);
     return { url, name };
 };

package/dist/commands/uninstall.js

@@ -18,6 +18,9 @@ export const usage = () => commandUsage({
             value: '<name>',
             description: 'Limit uninstall targets to workspace groups.',
         },
+        'lockfile-only': {
+            description: 'Only update lockfile and package.json files; skip node_modules operations.',
+        },
         'allow-scripts': {
             value: '<query>',
             description: 'Filter which packages are allowed to run lifecycle scripts using DSS query syntax.',
@@ -36,6 +39,10 @@ export const command = async (conf) => {
         String(conf.get('allow-scripts'))
         : ':not(*)';
     /* c8 ignore stop */
-    const { graph } = await uninstall({ ...conf.options, allowScripts }, remove);
+    const { graph } = await uninstall({
+        ...conf.options,
+        allowScripts,
+        lockfileOnly: conf.options['lockfile-only'],
+    }, remove);
     return { graph };
 };

package/dist/index.js

@@ -3,7 +3,7 @@ import { asRootError } from '@vltpkg/output/error';
 import { loadPackageJson } from 'package-json-from-dist';
 import { getSortedCliOptions, getSortedKeys, } from "./config/definition.js";
 import { Config } from "./config/index.js";
-import { outputCommand, stderr, stdout } from "./output.js";
+import { flushAndExit, outputCommand, stderr, stdout, } from "./output.js";
 import { indent } from "./print-err.js";
 import { loadCommand } from "./load-command.js";
 const { version } = loadPackageJson(import.meta.filename, process.env.__VLT_INTERNAL_CLI_PACKAGE_JSON);
@@ -35,7 +35,7 @@ const loadVlt = async (cwd, argv) => {
             stderr(indent(validOptions.join('\n'), 4));
         }
         stderr(indent(`Run 'vlt help' for more information about available options.`));
-        return process.exit(process.exitCode || 1);
+        return flushAndExit(process.exitCode || 1);
     }
 };
 const run = async () => {
@@ -64,7 +64,7 @@ const run = async () => {
         if (vlt.get('workspace-group')) {
             stderr(indent(`Workspace Group: ${format(vlt.get('workspace-group'))}`));
         }
-        return process.exit(process.exitCode || 1);
+        return flushAndExit(process.exitCode || 1);
     }
     if (vlt.command === 'registry') {
         const { dispatchRegistry } = await import("./commands/registry.js");

package/dist/mermaid-image-view.js

@@ -1,5 +1,5 @@
 import { mermaidOutput } from '@vltpkg/graph';
-import { urlOpen } from '@vltpkg/url-open';
+import { openUrl } from "./open-url.js";
 import { stderr } from "./output.js";
 import { ViewClass } from "./view.js";
 /**
@@ -24,13 +24,13 @@ export class MermaidImageView extends ViewClass {
             stderr(`Generating PNG image...`);
             const filePath = await renderMermaidToPng(mermaidText);
             stderr(`Image saved to: ${filePath}`);
-            await urlOpen(filePath);
+            await openUrl(filePath);
         }
         else {
             stderr(`Generating SVG image...`);
             const filePath = await renderMermaidToFile(mermaidText);
             stderr(`Image saved to: ${filePath}`);
-            await urlOpen(filePath);
+            await openUrl(filePath);
         }
     }
 }

package/dist/open-url.d.ts

@@ -0,0 +1,7 @@
+/**
+ * Open a url in the system browser, without failing the command if the
+ * opener does. The opener command may exist but still be unable to launch
+ * anything, e.g: `xdg-open` on a headless server. The url was already
+ * printed, so the user can open it manually.
+ */
+export declare const openUrl: (url: string) => Promise<void>;

package/dist/open-url.js

@@ -0,0 +1,16 @@
+import { urlOpen } from '@vltpkg/url-open';
+import { stderr } from "./output.js";
+/**
+ * Open a url in the system browser, without failing the command if the
+ * opener does. The opener command may exist but still be unable to launch
+ * anything, e.g: `xdg-open` on a headless server. The url was already
+ * printed, so the user can open it manually.
+ */
+export const openUrl = async (url) => {
+    try {
+        await urlOpen(url);
+    }
+    catch {
+        stderr(`Could not open a browser. Please open ${url} manually.`);
+    }
+};

package/dist/output.d.ts

@@ -4,6 +4,26 @@ import type { Command } from './index.ts';
 import type { View, Views } from './view.ts';
 export declare const stdout: (...args: unknown[]) => void;
 export declare const stderr: (...args: unknown[]) => void;
+/** the part of a stdio stream that {@link flushStream} waits on */
+export type FlushableStream = {
+    writableLength: number;
+    write: (chunk: string, cb: () => void) => boolean;
+};
+/**
+ * Resolve once everything already handed to `stream` has reached the
+ * other end. A zero length write is enough: writes complete in order,
+ * so its callback cannot run ahead of the ones already queued.
+ */
+export declare const flushStream: (stream: FlushableStream) => Promise<void>;
+/**
+ * Exit, but only once stdout and stderr have drained.
+ *
+ * Writes to a pipe are async, and `process.exit()` throws away whatever
+ * is still queued for one. Piped output is exactly what CI gives us, so
+ * exiting right after printing crops the output at the 64k pipe buffer,
+ * and since the error is printed last, the error is what gets lost.
+ */
+export declare const flushAndExit: (code?: Parameters<typeof process.exit>[0]) => Promise<never>;
 type StyleTextFn = (format: Parameters<typeof utilStyleText>[0], s: string) => string;
 export declare let styleTextStdout: StyleTextFn;
 export declare let styleTextStderr: StyleTextFn;

package/dist/output.js

@@ -27,6 +27,31 @@ const supportsColor = (stream) => {
 export const stdout = (...args) => console.log(...args);
 // eslint-disable-next-line no-console
 export const stderr = (...args) => console.error(...args);
+/**
+ * Resolve once everything already handed to `stream` has reached the
+ * other end. A zero length write is enough: writes complete in order,
+ * so its callback cannot run ahead of the ones already queued.
+ */
+export const flushStream = (stream) => new Promise(res => {
+    if (!stream.writableLength)
+        return res();
+    stream.write('', () => res());
+});
+/**
+ * Exit, but only once stdout and stderr have drained.
+ *
+ * Writes to a pipe are async, and `process.exit()` throws away whatever
+ * is still queued for one. Piped output is exactly what CI gives us, so
+ * exiting right after printing crops the output at the 64k pipe buffer,
+ * and since the error is printed last, the error is what gets lost.
+ */
+export const flushAndExit = async (code) => {
+    await Promise.all([
+        flushStream(process.stdout),
+        flushStream(process.stderr),
+    ]);
+    return process.exit(code);
+};
 /* c8 ignore start */
 const styleText = (f, s) => utilStyleText(f, s, { validateStream: false });
 /* c8 ignore stop */
@@ -202,6 +227,6 @@ export const outputCommand = async (cliCommand, conf, { start, vltVersion } = {
             ...formatOptions,
             colors: stderrColor,
         });
-        process.exit(process.exitCode);
+        await flushAndExit(process.exitCode);
     }
 };

package/dist/parse-add-remove-args.js

@@ -1,4 +1,4 @@
-import { asDependency } from '@vltpkg/graph';
+import { addKey, asDependency } from '@vltpkg/graph';
 import { joinDepIDTuple, splitDepID } from '@vltpkg/dep-id';
 import { Spec } from '@vltpkg/spec';
 import { isAbsolute, relative, resolve } from 'node:path';
@@ -85,10 +85,6 @@ export const parseAddArgs = (config, scurry, monorepo) => {
     const type = getType(config.values);
     const importers = getWorkspaceImporters(config.values, monorepo);
     const specOptions = config.options;
-    // nameless spec definitions will need to use their full
-    // stringified spec result instead of their name in order
-    // to have an unique key name in the resulting Map
-    const getName = (s) => s.name === '(unknown)' ? s.spec : s.name;
     // parses each positional argument into a Spec and
     // adds it to the new dependencies Map
     const parsedItems = [];
@@ -110,7 +106,7 @@ export const parseAddArgs = (config, scurry, monorepo) => {
             const importerDeps = new Map();
             for (const { spec, type: depType } of parsedItems) {
                 const rebased = rebaseFileSpec(spec, importerPath, scurryCwd, specOptions, monorepo);
-                importerDeps.set(getName(rebased), asDependency({ spec: rebased, type: depType }));
+                importerDeps.set(addKey(rebased), asDependency({ spec: rebased, type: depType }));
             }
             add.set(importer, importerDeps);
         }
@@ -118,7 +114,7 @@ export const parseAddArgs = (config, scurry, monorepo) => {
     else {
         const newDependencies = new Map();
         for (const { spec, type: depType } of parsedItems) {
-            newDependencies.set(getName(spec), asDependency({ spec, type: depType }));
+            newDependencies.set(addKey(spec), asDependency({ spec, type: depType }));
         }
         for (const importer of importers) {
             add.set(importer, newDependencies);

package/dist/query-auth.d.ts

@@ -0,0 +1,8 @@
+import type { GetAuthHeader } from '@vltpkg/query';
+import type { LoadedConfig } from './config/index.ts';
+/**
+ * Returns a {@link GetAuthHeader} implementation that looks up
+ * authorization tokens from the local keychain / env, used by
+ * network-reliant query selectors, e.g: `:outdated()`.
+ */
+export declare const createGetAuthHeader: (conf: LoadedConfig) => GetAuthHeader;

package/dist/query-auth.js

@@ -0,0 +1,7 @@
+import { getTokenByURL } from '@vltpkg/registry-client';
+/**
+ * Returns a {@link GetAuthHeader} implementation that looks up
+ * authorization tokens from the local keychain / env, used by
+ * network-reliant query selectors, e.g: `:outdated()`.
+ */
+export const createGetAuthHeader = (conf) => async (url) => getTokenByURL(url, conf.options.identity);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@vltpkg/cli-sdk",
   "description": "The source for the vlt CLI",
-  "version": "1.0.7",
+  "version": "1.0.8",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/vltpkg/vltpkg.git",
@@ -14,30 +14,30 @@
   },
   "dependencies": {
     "@resvg/resvg-wasm": "^2.6.2",
-    "@vltpkg/config": "1.0.7",
-    "@vltpkg/dep-id": "1.0.7",
-    "@vltpkg/dot-prop": "1.0.7",
-    "@vltpkg/error-cause": "1.0.7",
-    "@vltpkg/git": "1.0.7",
-    "@vltpkg/graph": "1.0.7",
-    "@vltpkg/graph-run": "1.0.7",
-    "@vltpkg/init": "1.0.7",
-    "@vltpkg/output": "1.0.7",
-    "@vltpkg/package-info": "1.0.7",
-    "@vltpkg/package-json": "1.0.7",
-    "@vltpkg/promise-spawn": "1.0.7",
-    "@vltpkg/query": "1.0.7",
-    "@vltpkg/registry-client": "1.0.7",
-    "@vltpkg/rollback-remove": "1.0.7",
-    "@vltpkg/run": "1.0.7",
-    "@vltpkg/security-archive": "1.0.7",
-    "@vltpkg/spec": "1.0.7",
-    "@vltpkg/types": "1.0.7",
-    "@vltpkg/url-open": "1.0.7",
-    "@vltpkg/vlt-json": "1.0.7",
-    "@vltpkg/vlx": "1.0.7",
-    "@vltpkg/workspaces": "1.0.7",
-    "@vltpkg/xdg": "1.0.7",
+    "@vltpkg/config": "1.0.8",
+    "@vltpkg/dep-id": "1.0.8",
+    "@vltpkg/dot-prop": "1.0.8",
+    "@vltpkg/error-cause": "1.0.8",
+    "@vltpkg/git": "1.0.8",
+    "@vltpkg/graph": "1.0.8",
+    "@vltpkg/graph-run": "1.0.8",
+    "@vltpkg/init": "1.0.8",
+    "@vltpkg/output": "1.0.8",
+    "@vltpkg/package-info": "1.0.8",
+    "@vltpkg/package-json": "1.0.8",
+    "@vltpkg/promise-spawn": "1.0.8",
+    "@vltpkg/query": "1.0.8",
+    "@vltpkg/registry-client": "1.0.8",
+    "@vltpkg/rollback-remove": "1.0.8",
+    "@vltpkg/run": "1.0.8",
+    "@vltpkg/security-archive": "1.0.8",
+    "@vltpkg/spec": "1.0.8",
+    "@vltpkg/types": "1.0.8",
+    "@vltpkg/url-open": "1.0.8",
+    "@vltpkg/vlt-json": "1.0.8",
+    "@vltpkg/vlx": "1.0.8",
+    "@vltpkg/workspaces": "1.0.8",
+    "@vltpkg/xdg": "1.0.8",
     "ansi-to-pre": "^1.0.6",
     "beautiful-mermaid": "^1.1.3",
     "chalk": "^5.6.2",