@oh-my-pi/pi-coding-agent 15.9.1 → 15.9.5

package/dist/types/modes/interactive-mode.d.ts

@@ -56,6 +56,7 @@ export declare class InteractiveMode implements InteractiveModeContext {
     todoContainer: Container;
     btwContainer: Container;
     omfgContainer: Container;
+    errorBannerContainer: Container;
     editor: CustomEditor;
     editorContainer: Container;
     hookWidgetContainerAbove: Container;
@@ -77,6 +78,7 @@ export declare class InteractiveMode implements InteractiveModeContext {
     todoPhases: TodoPhase[];
     hideThinkingBlock: boolean;
     pendingImages: ImageContent[];
+    pendingImageLinks: (string | undefined)[];
     compactionQueuedMessages: CompactionQueuedMessage[];
     pendingTools: Map<string, ToolExecutionHandle>;
     pendingBashComponents: BashExecutionComponent[];
@@ -136,6 +138,7 @@ export declare class InteractiveMode implements InteractiveModeContext {
     startPendingSubmission(input: {
         text: string;
         images?: ImageContent[];
+        imageLinks?: (string | undefined)[];
         customType?: string;
         display?: boolean;
     }): SubmittedUserInput;
@@ -162,6 +165,8 @@ export declare class InteractiveMode implements InteractiveModeContext {
         dim?: boolean;
     }): void;
     showError(message: string): void;
+    showPinnedError(message: string): void;
+    clearPinnedError(): void;
     showWarning(message: string): void;
     ensureLoadingAnimation(): void;
     setWorkingMessage(message?: string): void;
@@ -177,6 +182,7 @@ export declare class InteractiveMode implements InteractiveModeContext {
     isKnownSlashCommand(text: string): boolean;
     addMessageToChat(message: AgentMessage, options?: {
         populateHistory?: boolean;
+        imageLinks?: readonly (string | undefined)[];
     }): Component[];
     renderSessionContext(sessionContext: SessionContext, options?: {
         updateFooter?: boolean;
@@ -184,6 +190,7 @@ export declare class InteractiveMode implements InteractiveModeContext {
     }): void;
     renderInitialMessages(prebuiltContext?: SessionContext, options?: {
         preserveExistingChat?: boolean;
+        clearTerminalHistory?: boolean;
     }): void;
     getUserMessageText(message: Message): string;
     findLastAssistantMessage(): AssistantMessage | undefined;

package/dist/types/modes/types.d.ts

@@ -32,6 +32,7 @@ export type CompactionQueuedMessage = {
 export type SubmittedUserInput = {
     text: string;
     images?: ImageContent[];
+    imageLinks?: (string | undefined)[];
     customType?: string;
     display?: boolean;
     cancelled: boolean;
@@ -60,6 +61,7 @@ export interface InteractiveModeContext {
     todoContainer: Container;
     btwContainer: Container;
     omfgContainer: Container;
+    errorBannerContainer: Container;
     editor: CustomEditor;
     editorContainer: Container;
     hookWidgetContainerAbove: Container;
@@ -87,6 +89,7 @@ export interface InteractiveModeContext {
     planModePlanFilePath?: string;
     hideThinkingBlock: boolean;
     pendingImages: ImageContent[];
+    pendingImageLinks: (string | undefined)[];
     compactionQueuedMessages: CompactionQueuedMessage[];
     pendingTools: Map<string, ToolExecutionHandle>;
     pendingBashComponents: BashExecutionComponent[];
@@ -130,6 +133,8 @@ export interface InteractiveModeContext {
         dim?: boolean;
     }): void;
     showError(message: string): void;
+    showPinnedError(message: string): void;
+    clearPinnedError(): void;
     showWarning(message: string): void;
     showNewVersionNotification(newVersion: string): void;
     clearEditor(): void;
@@ -146,6 +151,7 @@ export interface InteractiveModeContext {
     startPendingSubmission(input: {
         text: string;
         images?: ImageContent[];
+        imageLinks?: (string | undefined)[];
         customType?: string;
         display?: boolean;
     }): SubmittedUserInput;
@@ -170,6 +176,7 @@ export interface InteractiveModeContext {
     isKnownSlashCommand(text: string): boolean;
     addMessageToChat(message: AgentMessage, options?: {
         populateHistory?: boolean;
+        imageLinks?: readonly (string | undefined)[];
     }): Component[];
     renderSessionContext(sessionContext: SessionContext, options?: {
         updateFooter?: boolean;

package/dist/types/modes/utils/ui-helpers.d.ts

@@ -24,6 +24,7 @@ export declare class UiHelpers {
     }): void;
     addMessageToChat(message: AgentMessage, options?: {
         populateHistory?: boolean;
+        imageLinks?: readonly (string | undefined)[];
     }): Component[];
     /**
      * Render session context to chat. Used for initial load and rebuild after compaction.

package/dist/types/session/agent-session.d.ts

@@ -201,6 +201,15 @@ export interface AgentSessionConfig {
      * **MUST NOT** dispose it on their own teardown.
      */
     ownedAsyncJobManager?: AsyncJobManager;
+    /**
+     * AsyncJobManager reachable by this session for scoped job actions.
+     *
+     * Top-level owners receive their own manager, subagents receive the inherited
+     * parent manager, and secondary in-process top-level sessions receive
+     * `undefined` so job snapshots and ACP drains cannot observe the primary's
+     * state.
+     */
+    asyncJobManager?: AsyncJobManager;
     /** Agent identity (registry id like "Main" or "Alice") used for IRC routing. */
     agentId?: string;
     /** Shared agent registry (for forwarding IRC observations to the main session UI). */

package/dist/types/session/auth-storage.d.ts

@@ -2,5 +2,5 @@
  * Re-exports from @oh-my-pi/pi-ai.
  * All credential storage types and the AuthStorage class now live in the ai package.
  */
-export type { ApiKeyCredential, AuthCredential, AuthCredentialEntry, AuthCredentialStore, AuthStorageData, AuthStorageOptions, OAuthCredential, SerializedAuthStorage, StoredAuthCredential, } from "@oh-my-pi/pi-ai";
-export { AuthBrokerClient, AuthStorage, REMOTE_REFRESH_SENTINEL, RemoteAuthCredentialStore, SqliteAuthCredentialStore, } from "@oh-my-pi/pi-ai";
+export type { ApiKeyCredential, AuthCredential, AuthCredentialEntry, AuthCredentialStore, AuthStorageData, AuthStorageOptions, OAuthCredential, SerializedAuthStorage, SnapshotResponse, StoredAuthCredential, } from "@oh-my-pi/pi-ai";
+export { AuthBrokerClient, AuthStorage, DEFAULT_SNAPSHOT_CACHE_TTL_MS, REMOTE_REFRESH_SENTINEL, RemoteAuthCredentialStore, readAuthBrokerSnapshotCache, SqliteAuthCredentialStore, writeAuthBrokerSnapshotCache, } from "@oh-my-pi/pi-ai";

package/dist/types/session/blob-store.d.ts

@@ -1,15 +1,16 @@
+export interface BlobPutOptions {
+    /** Optional file extension for a sidecar hardlink/copy that OS openers can type-detect. */
+    extension?: string;
+}
 export interface BlobPutResult {
     hash: string;
+    /** Canonical content-addressed path, always `<dir>/<sha256-hex>`. */
     path: string;
+    /** Path with the requested extension when supplied, otherwise the canonical path. */
+    displayPath: string;
     get ref(): string;
 }
-/**
- * Content-addressed blob store for externalizing large binary data (images) from session JSONL files.
- *
- * Files are stored at `<dir>/<sha256-hex>` with no extension. The SHA-256 hash is computed
- * over the raw binary data (not base64). Content-addressing makes writes idempotent and
- * provides automatic deduplication across sessions.
- */
+export declare function blobExtensionForImageMimeType(mimeType: string | undefined): string | undefined;
 export declare class BlobStore {
     readonly dir: string;
     constructor(dir: string);
@@ -17,13 +18,13 @@ export declare class BlobStore {
      * Write binary data to the blob store.
      * @returns SHA-256 hex hash of the data
      */
-    put(data: Buffer): Promise<BlobPutResult>;
+    put(data: Buffer, options?: BlobPutOptions): Promise<BlobPutResult>;
     /**
      * Synchronous variant of {@link put}. Use on persistence hot paths where the caller
      * cannot afford the microtask hops of the async version (e.g. OOM-safe session writes).
      * Returns once the bytes are in the kernel page cache.
      */
-    putSync(data: Buffer): BlobPutResult;
+    putSync(data: Buffer, options?: BlobPutOptions): BlobPutResult;
     /** Read blob by hash, returns Buffer or null if not found. */
     get(hash: string): Promise<Buffer | null>;
     /** Check if a blob exists. */
@@ -46,9 +47,9 @@ export declare function externalizeImageDataUrlSync(blobStore: BlobStore, dataUr
  * Externalize an image's base64 data to the blob store, returning a blob reference.
  * If the data is already a blob reference, returns it unchanged.
  */
-export declare function externalizeImageData(blobStore: BlobStore, base64Data: string): Promise<string>;
+export declare function externalizeImageData(blobStore: BlobStore, base64Data: string, mimeType?: string): Promise<string>;
 /** Synchronous variant of {@link externalizeImageData}. */
-export declare function externalizeImageDataSync(blobStore: BlobStore, base64Data: string): string;
+export declare function externalizeImageDataSync(blobStore: BlobStore, base64Data: string, mimeType?: string): string;
 /**
  * Resolve an externalized provider image data URL back to its original string.
  * If the data is not a blob reference, returns it unchanged.

package/dist/types/session/session-manager.d.ts

@@ -1,7 +1,7 @@
 import type { AgentMessage } from "@oh-my-pi/pi-agent-core";
 import type { ImageContent, Message, MessageAttribution, ServiceTier, TextContent } from "@oh-my-pi/pi-ai";
 import { ArtifactManager } from "./artifacts";
-import { type BlobPutResult } from "./blob-store";
+import { type BlobPutOptions, type BlobPutResult } from "./blob-store";
 import { type BashExecutionMessage, type CustomMessage, type FileMentionMessage, type HookMessage, type PythonExecutionMessage } from "./messages";
 import type { SessionStorage } from "./session-storage";
 export declare const CURRENT_SESSION_VERSION = 3;
@@ -208,7 +208,7 @@ export interface SessionInfo {
      */
     status?: SessionStatus;
 }
-export type ReadonlySessionManager = Pick<SessionManager, "getCwd" | "getSessionDir" | "getSessionId" | "getSessionFile" | "getSessionName" | "getArtifactsDir" | "getArtifactManager" | "allocateArtifactPath" | "saveArtifact" | "getArtifactPath" | "getLeafId" | "getLeafEntry" | "getEntry" | "getLabel" | "getBranch" | "getHeader" | "getEntries" | "getTree" | "getUsageStatistics" | "putBlob">;
+export type ReadonlySessionManager = Pick<SessionManager, "getCwd" | "getSessionDir" | "getSessionId" | "getSessionFile" | "getSessionName" | "getArtifactsDir" | "getArtifactManager" | "allocateArtifactPath" | "saveArtifact" | "getArtifactPath" | "getLeafId" | "getLeafEntry" | "getEntry" | "getLabel" | "getBranch" | "getHeader" | "getEntries" | "getTree" | "getUsageStatistics" | "putBlob" | "putBlobSync">;
 /** Exported for testing */
 export declare function migrateSessionEntries(entries: FileEntry[]): void;
 /** Exported for compaction.test.ts */
@@ -294,7 +294,9 @@ export declare class SessionManager {
     private readonly storage;
     private constructor();
     /** Puts a binary blob into the blob store and returns the blob reference */
-    putBlob(data: Buffer): Promise<BlobPutResult>;
+    putBlob(data: Buffer, options?: BlobPutOptions): Promise<BlobPutResult>;
+    /** Synchronous variant of {@link putBlob} for rebuild-only render paths. */
+    putBlobSync(data: Buffer, options?: BlobPutOptions): BlobPutResult;
     captureState(): SessionManagerStateSnapshot;
     restoreState(snapshot: SessionManagerStateSnapshot): void;
     /** Switch to a different session file (used for resume and branching) */

package/dist/types/system-prompt.d.ts

@@ -84,6 +84,8 @@ export interface BuildSystemPromptOptions {
     workspaceTree?: WorkspaceTree | Promise<WorkspaceTree>;
     /** Whether the local memory://root summary is active. */
     memoryRootEnabled?: boolean;
+    /** Active model identifier (e.g. "anthropic/claude-opus-4") surfaced to the agent. */
+    model?: string;
 }
 /** Result of building provider-facing system prompt messages. */
 export interface BuildSystemPromptResult {

package/dist/types/task/types.d.ts

@@ -149,6 +149,8 @@ export interface AgentDefinition {
     output?: unknown;
     blocking?: boolean;
     autoloadSkills?: string[];
+    /** When `false`, the agent's `read` tool returns verbatim file content instead of structural summaries. */
+    readSummarize?: boolean;
     source: AgentSource;
     filePath?: string;
 }

package/dist/types/tiny/title-client.d.ts

@@ -1,5 +1,19 @@
 import type { Subprocess } from "bun";
-import type { TinyTitleProgressEvent, TinyTitleWorkerOutbound } from "./title-protocol";
+import type { TinyTitleProgressEvent, TinyTitleWorkerInbound, TinyTitleWorkerOutbound } from "./title-protocol";
+/**
+ * Abstraction over the tiny-model subprocess. Modelled as a worker interface
+ * so existing callers (titles, memory completions, downloads) compose the
+ * same way; the runtime implementation is a Bun child process so
+ * `onnxruntime-node`'s NAPI finalizer never runs inside the main agent
+ * address space — that destructor segfaults Bun on Windows during shutdown
+ * (issue #1606).
+ */
+interface WorkerHandle {
+    send(message: TinyTitleWorkerInbound): void;
+    onMessage(handler: (message: TinyTitleWorkerOutbound) => void): () => void;
+    onError(handler: (error: Error) => void): () => void;
+    terminate(): Promise<void>;
+}
 export interface TinyTitleDownloadOptions {
     signal?: AbortSignal;
     onProgress?: (event: TinyTitleProgressEvent) => void;
@@ -39,6 +53,7 @@ interface SpawnedSubprocess {
 export declare function createTinyTitleSubprocess(): SpawnedSubprocess;
 export declare class TinyTitleClient {
     #private;
+    constructor(spawnWorker?: () => WorkerHandle);
     onProgress(listener: (event: TinyTitleProgressEvent) => void): () => void;
     generate(modelKey: string, message: string, signal?: AbortSignal): Promise<string | null>;
     complete(modelKey: string, prompt: string, options?: {

package/dist/types/tool-discovery/mode.d.ts

@@ -0,0 +1,8 @@
+import type { Settings } from "../config/settings";
+import type { SettingValue } from "../config/settings-schema";
+export declare const TOOL_DISCOVERY_AUTO_THRESHOLD = 40;
+export declare const TOOL_DISCOVERY_SEARCH_TOOL_NAME = "search_tool_bm25";
+export type ToolDiscoveryModeSetting = SettingValue<"tools.discoveryMode">;
+export type EffectiveToolDiscoveryMode = Exclude<ToolDiscoveryModeSetting, "auto">;
+export declare function countToolsForAutoDiscovery(toolNames: Iterable<string>): number;
+export declare function resolveEffectiveToolDiscoveryMode(settings: Settings, toolCount: number): EffectiveToolDiscoveryMode;

package/dist/types/tools/archive-reader.d.ts

@@ -21,7 +21,11 @@ interface TarStorage {
 }
 interface ZipStorage {
     type: "zip";
-    bytes: Uint8Array;
+    archivePath: string;
+    compressedSize: number;
+    compression: number;
+    flags: number;
+    localHeaderOffset: number;
 }
 type EntryStorage = TarStorage | ZipStorage;
 interface ArchiveIndexEntry extends ArchiveNode {

package/dist/types/tools/index.d.ts

@@ -1,6 +1,7 @@
 import type { InMemorySnapshotStore } from "@oh-my-pi/hashline";
 import type { AgentTelemetryConfig, AgentTool } from "@oh-my-pi/pi-agent-core";
 import type { ToolChoice } from "@oh-my-pi/pi-ai";
+import type { AsyncJobManager } from "../async/job-manager";
 import type { PromptTemplate } from "../config/prompt-templates";
 import type { Settings } from "../config/settings";
 import type { Skill } from "../extensibility/skills";
@@ -139,6 +140,21 @@ export interface ToolSession {
     modelRegistry?: import("../config/model-registry").ModelRegistry;
     /** Agent output manager for unique agent:// IDs across task invocations */
     agentOutputManager?: AgentOutputManager;
+    /**
+     * Async job manager scoped to this session.
+     *
+     * - Top-level session that constructed one: its own manager.
+     * - Subagent (`parentTaskPrefix` set): the parent's manager, so background
+     *   bash/task work and `onJobComplete` deliveries flow into the conversation
+     *   that spawned it.
+     * - Secondary in-process top-level session that found a singleton already
+     *   installed (issue #1923): `undefined`. Tools refuse async work rather
+     *   than silently route completions into the owning session's `yieldQueue`.
+     *
+     * Tools MUST use this instead of `AsyncJobManager.instance()` so a secondary
+     * session never borrows the owning session's manager by accident.
+     */
+    asyncJobManager?: AsyncJobManager;
     /** MCP manager visible to subagents without relying on the process-global singleton. */
     mcpManager?: MCPManager;
     /** Local protocol root to propagate to nested subagents and eval-created agents. */

package/dist/types/tools/path-utils.d.ts

@@ -17,6 +17,17 @@ export declare function parseLineRangeChunk(sel: string): LineRange | null;
  * downstream consumers can stream the file in a single forward pass per range.
  */
 export declare function parseLineRanges(sel: string): [LineRange, ...LineRange[]] | null;
+/**
+ * Extract the line-range component from a read-tool selector that may also
+ * carry a verbatim/index display mode (`raw`, `conflicts`) — alone or compounded
+ * with a range (`raw:50-100`, `50-100:raw`). Returns the parsed ranges when the
+ * selector names any, otherwise `undefined` (pure `raw`/`conflicts`/none).
+ *
+ * Used by content search, which honors line ranges as a match filter but has no
+ * use for verbatim/conflict display modes — so those selectors are accepted and
+ * treated as an unfiltered, whole-resource search rather than rejected.
+ */
+export declare function selectorLineRanges(sel: string | undefined): [LineRange, ...LineRange[]] | undefined;
 /** Return `true` when `lineNumber` (1-indexed) falls in any of the supplied ranges. */
 export declare function isLineInRanges(lineNumber: number, ranges: readonly LineRange[]): boolean;
 export declare function splitPathAndSel(rawPath: string): {

package/dist/types/tui/hyperlink.d.ts

@@ -7,6 +7,18 @@
  * - `"always"`: unconditionally (useful for viewers that support OSC 8 without advertising it)
  */
 export declare function isHyperlinkEnabled(): boolean;
+/**
+ * Wrap `displayText` in an OSC 8 hyperlink pointing at `uri`.
+ *
+ * Returns `displayText` unchanged when hyperlinks are disabled, `uri` contains
+ * terminal control bytes, or `displayText` already contains an OSC 8 sequence.
+ */
+export declare function uriHyperlink(uri: string, displayText: string): string;
+/**
+ * Wrap `displayText` in an OSC 8 hyperlink pointing at an HTTP(S) URL.
+ * `www.example.com` inputs are linked as `https://www.example.com`.
+ */
+export declare function urlHyperlink(url: string, displayText: string): string;
 /**
  * Wrap `displayText` in an OSC 8 hyperlink pointing at the given absolute file path.
  *

package/dist/types/web/search/render.d.ts

@@ -5,7 +5,7 @@
  */
 import type { Component } from "@oh-my-pi/pi-tui";
 import type { RenderResultOptions } from "../../extensibility/custom-tools/types";
-import type { Theme } from "../../modes/theme/theme";
+import { type Theme } from "../../modes/theme/theme";
 import type { SearchResponse } from "./types";
 export interface SearchRenderDetails {
     response: SearchResponse;
@@ -20,7 +20,6 @@ export declare function renderSearchResult(result: {
     details?: SearchRenderDetails;
 }, options: RenderResultOptions, theme: Theme, args?: {
     query?: string;
-    allowLongAnswer?: boolean;
     maxAnswerLines?: number;
 }): Component;
 /** Render web search call (query preview) */

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "15.9.1",
+	"version": "15.9.5",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -47,14 +47,14 @@
 		"@agentclientprotocol/sdk": "0.22.1",
 		"@babel/parser": "^7.29.7",
 		"@mozilla/readability": "^0.6.0",
-		"@oh-my-pi/hashline": "15.9.1",
-		"@oh-my-pi/omp-stats": "15.9.1",
-		"@oh-my-pi/pi-agent-core": "15.9.1",
-		"@oh-my-pi/pi-ai": "15.9.1",
-		"@oh-my-pi/pi-mnemopi": "15.9.1",
-		"@oh-my-pi/pi-natives": "15.9.1",
-		"@oh-my-pi/pi-tui": "15.9.1",
-		"@oh-my-pi/pi-utils": "15.9.1",
+		"@oh-my-pi/hashline": "15.9.5",
+		"@oh-my-pi/omp-stats": "15.9.5",
+		"@oh-my-pi/pi-agent-core": "15.9.5",
+		"@oh-my-pi/pi-ai": "15.9.5",
+		"@oh-my-pi/pi-mnemopi": "15.9.5",
+		"@oh-my-pi/pi-natives": "15.9.5",
+		"@oh-my-pi/pi-tui": "15.9.5",
+		"@oh-my-pi/pi-utils": "15.9.5",
 		"@opentelemetry/api": "^1.9.1",
 		"@opentelemetry/context-async-hooks": "^2.7.1",
 		"@opentelemetry/exporter-trace-otlp-proto": "^0.218.0",

package/src/cli/classify-install-target.ts

@@ -1,7 +1,11 @@
 /**
- * Classify an install spec as a marketplace plugin reference or a plain npm package.
+ * Classify an install spec as a local path, marketplace plugin reference, or
+ * plain npm package.
  *
  * Rules (applied in order):
+ *  0. Looks like a filesystem path (`.`, `..`, `./…`, `..\…`, `/…`, `~/…`,
+ *     `C:\…`, `\\unc`) -> local. Routed through `PluginManager.link()` so the
+ *     `omp plugin install <path>` and `omp plugin link <path>` flows agree.
  *  1. Starts with `@` (scoped npm) -> always npm.
  *  2. Contains `@` after the first character -> split on the LAST `@`.
  *     If the right-hand side is a known marketplace name, it's a marketplace ref.
@@ -25,10 +29,32 @@ const NPM_DIST_TAGS = new Set([
 // Semver-like: starts with digit, or contains version range prefixes
 const LOOKS_LIKE_VERSION = /^[\d~^>=<]/;
 
-export function classifyInstallTarget(
-	spec: string,
-	knownMarketplaces: Set<string>,
-): { type: "marketplace"; name: string; marketplace: string } | { type: "npm"; spec: string } {
+/**
+ * Detect specs that name a filesystem path rather than a package: bare `.` /
+ * `..`, cwd-relative (`./`, `../`, `.\`, `..\`), absolute (`/`, `C:\`, `C:/`,
+ * UNC `\\`), and tilde-prefixed (`~`, `~/`, `~\`). Tilde paths still rely on
+ * the shell or the caller for expansion — we only classify them so they reach
+ * the link path instead of npm-name validation.
+ */
+function isLocalPathSpec(spec: string): boolean {
+	if (spec === "." || spec === ".." || spec === "~") return true;
+	if (spec.startsWith("./") || spec.startsWith("../")) return true;
+	if (spec.startsWith(".\\") || spec.startsWith("..\\")) return true;
+	if (spec.startsWith("~/") || spec.startsWith("~\\")) return true;
+	if (spec.startsWith("/")) return true;
+	if (spec.startsWith("\\\\")) return true;
+	if (/^[A-Za-z]:[\\/]/.test(spec)) return true;
+	return false;
+}
+
+export type ClassifiedInstallTarget =
+	| { type: "local"; path: string }
+	| { type: "marketplace"; name: string; marketplace: string }
+	| { type: "npm"; spec: string };
+
+export function classifyInstallTarget(spec: string, knownMarketplaces: Set<string>): ClassifiedInstallTarget {
+	// Rule 0: filesystem path — bypass npm/marketplace validation entirely.
+	if (isLocalPathSpec(spec)) return { type: "local", path: spec };
 	// Rule 1: scoped npm package — @ at position 0 is never a marketplace separator.
 	if (spec.startsWith("@")) return { type: "npm", spec };
 	// Rule 2: @ somewhere after the first character.