From 0e17c955a65809f88673ea56d722a9ff1d6b8236 Mon Sep 17 00:00:00 2001 From: Sawyer Date: Thu, 17 Sep 2026 16:29:53 -0700 Subject: [PATCH] chore(web,tools,scripts): drop ticket references from comments Linear ticket tags and PR references in code comments create confusion as things change; git history already carries that context. Strips CL-####/INTR-###/PR-# references from comments, test titles, and log strings across apps/web, tools, agents, skills, scripts, and test, and trims or deletes comments that only narrated history once the tag is gone. --- agents/assistant/src/index.ts | 4 +- agents/assistant/src/system-prompt.ts | 14 +-- .../test/manager-tools-scenario.test.ts | 13 +- apps/web/README.md | 2 +- apps/web/src/agent-chat-launch.ts | 6 +- apps/web/src/agent-dm-launch.ts | 2 +- apps/web/src/agents-api.ts | 6 +- apps/web/src/agents-directory.ts | 4 +- apps/web/src/api.ts | 4 +- apps/web/src/app-error-boundary.tsx | 2 +- apps/web/src/app.css | 36 +++--- apps/web/src/auto-workbench-title.test.ts | 4 +- apps/web/src/auto-workbench-title.ts | 2 +- apps/web/src/chat-artifact-open.ts | 2 +- apps/web/src/client-bootstrap.ts | 2 +- apps/web/src/command-palette-actions.test.ts | 4 +- apps/web/src/command-palette-actions.ts | 10 +- apps/web/src/command-palette-provider.tsx | 10 +- apps/web/src/connect-service-actions.ts | 6 +- apps/web/src/granola-plugin-availability.ts | 2 +- apps/web/src/insights-api.test.ts | 4 +- apps/web/src/insights-api.ts | 18 +-- apps/web/src/insights-deeplinks.ts | 2 +- apps/web/src/insights-path.ts | 2 +- apps/web/src/insights-stats.test.ts | 4 +- apps/web/src/insights-stats.ts | 12 +- apps/web/src/insights-workbench-scope.ts | 4 +- apps/web/src/instant-agent-create.test.ts | 2 +- apps/web/src/instant-agent-create.ts | 4 +- apps/web/src/last-workbench.ts | 2 +- apps/web/src/library-workbench-scope.ts | 2 +- apps/web/src/login-next.test.ts | 2 +- apps/web/src/login-next.ts | 2 +- apps/web/src/main.tsx | 4 +- apps/web/src/myra-deploy.ts | 2 +- apps/web/src/myra-source.ts | 4 +- apps/web/src/myra-workbench.ts | 2 +- apps/web/src/navigation.tsx | 2 +- apps/web/src/needs-converge.ts | 2 +- apps/web/src/onboarding.ts | 20 +-- .../src/onboarding/provider-connect-step.tsx | 4 +- apps/web/src/pages/agent-detail-page.tsx | 2 +- apps/web/src/pages/agents-page.tsx | 25 ++-- apps/web/src/pages/chat-page.tsx | 14 +-- apps/web/src/pages/create-agent-panel.tsx | 23 ++-- apps/web/src/pages/create-skill-dialog.tsx | 2 +- apps/web/src/pages/home-page.tsx | 17 ++- .../src/pages/insights-page-render.test.tsx | 2 +- apps/web/src/pages/insights-page.test.ts | 2 +- apps/web/src/pages/insights-page.tsx | 20 +-- .../src/pages/legacy-settings-redirects.tsx | 6 +- apps/web/src/pages/library-page.tsx | 12 +- apps/web/src/pages/mission-control-page.tsx | 6 +- apps/web/src/pages/new-workbench-picker.tsx | 6 +- apps/web/src/pages/onboarding-page.tsx | 2 +- apps/web/src/pages/plugins-page.tsx | 10 +- .../src/pages/routines-available-strings.ts | 2 +- apps/web/src/pages/routines-page.tsx | 2 +- .../src/pages/run-status-tone-parity.test.ts | 2 +- apps/web/src/pages/skill-detail-page.tsx | 12 +- apps/web/src/pages/skills-page.tsx | 12 +- apps/web/src/pages/workbench-timeline.tsx | 2 +- apps/web/src/pages/workflow-detail-page.tsx | 13 +- apps/web/src/path-ids.ts | 8 +- apps/web/src/profile-relations.ts | 2 +- apps/web/src/query-client.test.ts | 2 +- apps/web/src/query-client.ts | 6 +- apps/web/src/routes.tsx | 40 +++--- apps/web/src/routines-api.ts | 18 +-- apps/web/src/settings-groups.ts | 2 +- apps/web/src/shell/app-shell.tsx | 2 +- apps/web/src/shell/bench-activity.ts | 2 +- apps/web/src/shell/brand-mark.tsx | 11 +- apps/web/src/shell/canvas-availability.tsx | 8 +- apps/web/src/shell/canvas-column.tsx | 12 +- apps/web/src/shell/composer-insertion.tsx | 2 +- .../web/src/shell/context-menu/items.test.tsx | 6 +- apps/web/src/shell/context-menu/items.tsx | 2 +- apps/web/src/shell/context-menu/targets.ts | 2 +- apps/web/src/shell/docks.tsx | 2 +- apps/web/src/shell/library-artifacts.ts | 6 +- apps/web/src/shell/provider-health-banner.tsx | 12 +- .../src/shell/provider-health-context.test.ts | 4 +- .../web/src/shell/provider-health-context.tsx | 18 +-- apps/web/src/shell/routine-activity.ts | 4 +- apps/web/src/shell/sidebar-rows.test.ts | 10 +- apps/web/src/shell/sidebar-rows.ts | 6 +- apps/web/src/shell/sidebar.tsx | 13 +- apps/web/src/shell/workbench-list.test.ts | 4 +- apps/web/src/shell/workbench-list.tsx | 2 +- apps/web/src/skill-display-name.ts | 4 +- apps/web/src/skills-api.ts | 6 +- apps/web/src/workflow-detail-api.ts | 13 +- apps/web/test/agent-detail-page.test.tsx | 4 +- apps/web/test/agent-skills-picker.test.tsx | 2 +- apps/web/test/agents-api.test.ts | 6 +- apps/web/test/agents-page.test.tsx | 28 ++--- apps/web/test/app-error-boundary.test.tsx | 4 +- apps/web/test/auth.test.tsx | 6 +- apps/web/test/bench-activity.test.tsx | 6 +- apps/web/test/canvas-column.test.tsx | 4 +- apps/web/test/create-agent-panel.test.tsx | 17 ++- apps/web/test/fetch-dedupe.test.tsx | 6 +- apps/web/test/home-page.test.tsx | 18 +-- apps/web/test/insights-route.test.tsx | 8 +- .../test/legacy-settings-redirects.test.tsx | 6 +- .../test/library-artifact-deep-link.test.tsx | 2 +- apps/web/test/library-page-selection.test.tsx | 2 +- .../test/library-upload-confirmation.test.tsx | 2 +- apps/web/test/login-routing.test.tsx | 2 +- apps/web/test/mission-control-page.test.tsx | 4 +- apps/web/test/pages.test.tsx | 6 +- apps/web/test/plugins-page.test.tsx | 33 +++-- apps/web/test/provider-health-banner.test.tsx | 14 +-- .../web/test/provisioning-error-page.test.tsx | 2 +- apps/web/test/query-client.test.ts | 2 +- apps/web/test/routes.test.tsx | 32 +++-- apps/web/test/routine-activity.test.ts | 2 +- apps/web/test/routine-detail-page.test.tsx | 2 +- apps/web/test/routines-page.test.tsx | 4 +- apps/web/test/settings-access.test.tsx | 2 +- apps/web/test/shell-chrome-wiring.test.tsx | 4 +- apps/web/test/sidebar.test.tsx | 10 +- apps/web/test/skill-detail-page.test.tsx | 4 +- apps/web/test/skills-page.test.tsx | 4 +- .../test/stage-chrome-consistency.test.tsx | 4 +- apps/web/test/stage-search-filter.test.tsx | 2 +- apps/web/test/stage-top-nav.test.tsx | 2 +- apps/web/test/toast-single-system.test.tsx | 8 +- apps/web/test/workbench-list.test.tsx | 2 +- apps/web/test/workflow-detail-page.test.tsx | 2 +- scripts/db-setup.ts | 10 +- scripts/e2e/browser/walkthrough.ts | 36 +++--- scripts/e2e/db-gate.test.ts | 2 +- scripts/e2e/db-gate.ts | 2 +- scripts/e2e/docker-sidecar-smoke.ts | 2 +- scripts/e2e/harness.test.ts | 4 +- scripts/e2e/harness.ts | 8 +- scripts/e2e/noop-inference-server.ts | 14 +-- scripts/e2e/smoke-auth.test.ts | 2 +- scripts/e2e/smoke-chat.test.ts | 2 +- scripts/e2e/smoke-onboarding.test.ts | 2 +- scripts/e2e/smoke-webhook.test.ts | 119 +++++++++--------- scripts/e2e/t3-github-roundtrip.proof.test.ts | 2 +- scripts/e2e/walking-skeleton.test.ts | 4 +- scripts/lib/publishable-packages.ts | 2 +- scripts/repro/connect-github-smoke.ts | 12 +- skills/skills/src/skill-md.ts | 4 +- test/isolation/isolation.test.ts | 7 +- tools/access/src/client.ts | 2 +- tools/agent-directory/src/client.test.ts | 3 +- tools/agent-directory/src/client.ts | 4 +- tools/agent-directory/src/tool.ts | 2 +- tools/capability/README.md | 8 +- tools/capability/package.json | 2 +- tools/capability/src/client.ts | 29 ++--- tools/capability/src/tool.ts | 6 +- tools/catalog/README.md | 2 +- tools/catalog/src/index.ts | 2 +- tools/catalog/src/tool.test.ts | 2 +- tools/catalog/src/tool.ts | 2 +- tools/connections/src/tool.test.ts | 2 +- tools/connections/src/tool.ts | 14 +-- tools/github/README.md | 2 +- tools/github/src/manifest.test.ts | 2 +- tools/github/src/repos.ts | 6 +- tools/granola/README.md | 8 +- tools/granola/src/manifest.test.ts | 2 +- tools/granola/src/tool.ts | 4 +- .../test/credential-delivery.drizzle.test.ts | 2 +- tools/interaction/src/client.ts | 2 +- tools/interaction/src/tool.ts | 2 +- tools/linear/README.md | 6 +- tools/linear/src/manifest.test.ts | 2 +- tools/linear/src/tool.ts | 4 +- .../test/credential-delivery.drizzle.test.ts | 2 +- .../credential-wiring-e2e.drizzle.test.ts | 2 +- tools/mcp/src/allowance.ts | 2 +- tools/mcp/src/tool.ts | 2 +- tools/reddit/README.md | 2 +- tools/reddit/src/manifest.test.ts | 2 +- tools/skills-tools/src/client.test.ts | 2 +- tools/skills-tools/src/client.ts | 12 +- tools/skills-tools/src/tool.test.ts | 2 +- tools/skills-tools/src/tool.ts | 2 +- tools/tools-skills/src/client.test.ts | 2 +- tools/tools-skills/src/client.ts | 4 +- tools/tools-skills/src/tool.test.ts | 2 +- tools/web-search/README.md | 2 +- tools/web-search/src/manifest.test.ts | 2 +- tools/workflow-authoring/README.md | 6 +- tools/workflow-authoring/src/client.ts | 2 +- tools/workflow-authoring/src/tool.ts | 5 +- 193 files changed, 633 insertions(+), 704 deletions(-) diff --git a/agents/assistant/src/index.ts b/agents/assistant/src/index.ts index 773a825644..bc2e799063 100644 --- a/agents/assistant/src/index.ts +++ b/agents/assistant/src/index.ts @@ -9,13 +9,13 @@ // platform's deploy machinery; the execution host materializes it at // runtime from the deploy alone. // -// Tool-package pins (CL-5999, CL-5852): `@intx/agent`'s `defineAgent` +// Tool-package pins: `@intx/agent`'s `defineAgent` // still does not accept a `toolPackagePins` field on its authoring-time // config — it is vendored, read-only source for this change — so the // agent below is built directly against `AgentDefinition`'s own type, // which already carries the field, matching // `workflows/collateral-generation`'s precedent. `@corbits/memory` -// (CL-8186) is pinned so this deployment can search, add, and list the +// is pinned so this deployment can search, add, and list the // tenant's firm memory (`memory_search`/`memory_add`/`memory_list`) — a // published dependency, not a workbench-built tool package, resolved // through its own `interchange.tools` surface the same way any other diff --git a/agents/assistant/src/system-prompt.ts b/agents/assistant/src/system-prompt.ts index 16f6064812..e8a6fb955a 100644 --- a/agents/assistant/src/system-prompt.ts +++ b/agents/assistant/src/system-prompt.ts @@ -1,7 +1,7 @@ // Myra's pure prompt/pin data — no runtime import of `@intx/workflow` or // `@intx/agent` (only a type-only import below), so this module is safe -// to bundle into a browser build (CL-8154's `apps/web` needs exactly -// this: the content, not `buildAssistantWorkflow`'s `defineWorkflow` +// to bundle into a browser build (`apps/web` needs exactly this: the +// content, not `buildAssistantWorkflow`'s `defineWorkflow` // call, which pulls in `@intx/agent`'s Node-bound runtime — file // locking, in particular — that a browser bundle cannot resolve). // `index.ts`'s `buildAssistantWorkflow` imports these same constants; @@ -18,7 +18,7 @@ export const ASSISTANT_STEP_ID = "assistant"; * these: `@corbits/memory` is a normal published dependency, not a * workspace tool package the pin/registry resolver can name, so its * tool factories are attached directly on `toolFactories` in - * `./index.ts` instead (CL-8186). `@corbits/capability-tools` lets Myra + * `./index.ts` instead. `@corbits/capability-tools` lets Myra * self-service a missing tool, skill, or model; the manager-tools * bundles give Myra real workbench-management capability — a specialist * agent she can create (each gets their own chat), connection @@ -63,7 +63,7 @@ const ASSISTANT_WELCOME_CLAUSE = "allowed to."; /** - * TRIAGE (CL-6350): how Myra decides, on every message, whether to + * TRIAGE: how Myra decides, on every message, whether to * answer directly or delegate — what to do when a job needs a * connection this workbench doesn't have yet, and the reverse case: she * answers by default and only steps back when a message @-mentions a @@ -95,14 +95,14 @@ const ASSISTANT_TRIAGE_CLAUSE = "unless you are @-mentioned too or the sender asks you directly."; /** - * TEAMMATE (CL-7469 folds in COORDINATOR): how Myra offers help without + * TEAMMATE (folds in COORDINATOR): how Myra offers help without * pushing it — folds in the skills-capture nudge rather than a separate * always-on clause. Myra is the coordinator of this bench's agents, not * just its resident assistant: she never builds a teammate by * hand-editing files or answering in its place, and never stands up a * second instance of an agent that already exists. * - * The build-arc sentence (CL-5879) is the load-bearing one: it never + * The build-arc sentence is the load-bearing one: it never * waits for someone to name the mechanism ("make an agent", "set up a * routine"). An *outcome* — a sales motion, a content pipeline, a repo * to keep up — is enough on its own for Myra to work out the @@ -145,7 +145,7 @@ const ASSISTANT_TEAMMATE_CLAUSE = "built and what it can now do once it's live."; /** - * DISCOVERY (CL-6179): the specific interview-then-propose procedure a + * DISCOVERY: the specific interview-then-propose procedure a * stated outcome triggers, on top of the general "work out the team * yourself" doctrine in `ASSISTANT_TRIAGE_CLAUSE` above — short and * concrete rather than open-ended, so discovery never turns into a diff --git a/agents/assistant/test/manager-tools-scenario.test.ts b/agents/assistant/test/manager-tools-scenario.test.ts index cb963d8f0b..0fc7c6fc85 100644 --- a/agents/assistant/test/manager-tools-scenario.test.ts +++ b/agents/assistant/test/manager-tools-scenario.test.ts @@ -1,5 +1,4 @@ -// A scripted playback of the owner's canonical GTM scenario (CL-5879 -// manager-tools follow-up), exercised directly against the manager-tools +// A scripted playback of the owner's canonical GTM scenario, exercised directly against the manager-tools // bundles' `run()` with a single fake `fetch` standing in for every // workflow-run-authenticated route those bundles call. This proves the // CONTRACT across packages — a tool call's request shape matches what @@ -132,9 +131,9 @@ function createFakeHub() { } if (url.pathname === "/api/workflow-chat/participants/messages" && method === "POST" && body) { - // The connect-service card post `request_connection` (CL-6393) - // makes into the caller's own room. Recorded so the scenario can - // assert the card itself, not just the tool's advice text. + // The connect-service card post `request_connection` makes into + // the caller's own room. Recorded so the scenario can assert the + // card itself, not just the tool's advice text. const parts = body["parts"] as { kind: string; block: { type: string; data: Record }; @@ -244,7 +243,7 @@ async function runScenario(hub: ReturnType): Promise // A connector name that is neither a fixed `CONNECTOR_REGISTRY` // entry, a curated MCP preset, nor a connected MCP server makes // `request_connection` tell the agent to keep helping with what it - // can do (CL-6393) — never "go add a server and report back". + // can do — never "go add a server and report back". const unknown = await connectionsBundle.run( call("req_unknown", REQUEST_CONNECTION_TOOL, { connector: "acmecrm" }), new AbortController().signal, @@ -255,7 +254,7 @@ async function runScenario(hub: ReturnType): Promise expect(hub.postedCards).toHaveLength(0); // She puts a connect card in the room for each of the three (real) - // services (CL-6393) — the card is what the human clicks; the tool + // services — the card is what the human clicks; the tool // result only tells Myra to keep helping in the meantime. for (const connector of ["exa", "granola", "linear"]) { const result = await connectionsBundle.run( diff --git a/apps/web/README.md b/apps/web/README.md index 1c612f34c4..349f7e8952 100644 --- a/apps/web/README.md +++ b/apps/web/README.md @@ -57,7 +57,7 @@ Running routines in the contextual column are sourced today from | `/c` | Workbench deep-link surface. On wide layouts the conversation opens in the right canvas; on compact layouts it fills the main pane. Legacy `/chat` links still resolve here. | | `/workflows` | Workflow runs executing across your benches. | | `/library` | The artifact gallery. See "Library" below. | -| `/settings` | Account, bench membership, and (CL-5990) Agents/Skills. See "Agents" and "Skills" below. | +| `/settings` | Account, bench membership, and Agents/Skills. See "Agents" and "Skills" below. | Agents and Skills are Settings sections, not rail destinations — old `/agents[/:id]` and `/skills[/:id]` links still resolve, redirecting to diff --git a/apps/web/src/agent-chat-launch.ts b/apps/web/src/agent-chat-launch.ts index cafb0ebef1..74ec787595 100644 --- a/apps/web/src/agent-chat-launch.ts +++ b/apps/web/src/agent-chat-launch.ts @@ -2,9 +2,9 @@ // fresh chat with it" — the same `POST /workbenches` call this app's every // create path uses. `CreateAgentPanel`'s Settings → Agents entry point // calls this on success so an explicitly-defined new agent never ends -// nowhere, and `instant-agent-create.ts` — THE one creation verb -// (CL-6138) — calls it against the account's default setup template. -// Always creates (CL-6089) — never the `reuseExisting` land-hop path, +// nowhere, and `instant-agent-create.ts` — THE one creation verb — +// calls it against the account's default setup template. +// Always creates — never the `reuseExisting` land-hop path, // which is `default-agent-workbench.ts`'s own call, not this one. import { createWorkbench } from "@corbits/chat-ui"; diff --git a/apps/web/src/agent-dm-launch.ts b/apps/web/src/agent-dm-launch.ts index 7bc9fcec63..4d28fa0bda 100644 --- a/apps/web/src/agent-dm-launch.ts +++ b/apps/web/src/agent-dm-launch.ts @@ -1,5 +1,5 @@ // The one path from an agent definition's id to "the person is in their -// direct chat with it" (CL-6253) — the sidebar's agent rows are the one +// direct chat with it" — the sidebar's agent rows are the one // caller. Mirrors `agent-chat-launch.ts`'s shape exactly, but through // `openAgentDm` (`kind: "chat"`, `reuseExisting: true`) rather than // `createWorkbench` directly: the first click mints the DM, every later diff --git a/apps/web/src/agents-api.ts b/apps/web/src/agents-api.ts index 7e402aef0f..b76f4446b7 100644 --- a/apps/web/src/agents-api.ts +++ b/apps/web/src/agents-api.ts @@ -119,7 +119,7 @@ export function listAgentInstances(tenantId: string): Promise { @@ -160,7 +160,7 @@ export type AgentDefinitionDraft = typeof AgentDefinitionDraftResponse.infer.dra * Asks Myra to draft a starting system prompt (and optionally a * refined description, a model pick, and skills) from a name and a * plain-language purpose — the create-agent panel's "Create & chat" - * flow (CL-6074). Hits `@corbits/agent-directory`'s + * flow. Hits `@corbits/agent-directory`'s * `POST .../planner/agent-definitions/draft`; never deploys anything * itself. A caller that gets a rejected promise here (Myra unavailable, * the draft timing out, an unparseable or out-of-inventory reply, or a diff --git a/apps/web/src/agents-directory.ts b/apps/web/src/agents-directory.ts index bb543cf1a8..d045112485 100644 --- a/apps/web/src/agents-directory.ts +++ b/apps/web/src/agents-directory.ts @@ -18,7 +18,7 @@ import { import type { AgentDefinition, AgentInstance } from "./agents-api"; -/** An `AgentDefinition` with its display name derived (CL-6413) — its own +/** An `AgentDefinition` with its display name derived — its own * description when the definition was created with one, otherwise a * humanized reading of its immutable `name` slug. `name` itself stays the * slug throughout; nothing here mutates it. */ @@ -41,7 +41,7 @@ export function purposeAgentDefinitions( * (see `./agents-api.ts`), which excludes every non-top-level run * (workbench host, invited agent, task) server-side — the native * `GET /workflows/runs` listing's own predicate. That listing is - * single-tenant (CL-8087 accepted loss: no descendant-subtree rollup), + * single-tenant (accepted loss: no descendant-subtree rollup), * so this still applies the shared name-based workbench-host filter as * defense in depth. */ diff --git a/apps/web/src/api.ts b/apps/web/src/api.ts index f541f8d065..f7e4b97693 100644 --- a/apps/web/src/api.ts +++ b/apps/web/src/api.ts @@ -53,8 +53,6 @@ export const ArtifactRowSchema = ArtifactListItemSchema.merge( content: "string", }), ); -// GET /api/tenants/:id/artifacts/:artifactId wraps the row in an `artifact` -// envelope key (`@corbits/artifacts`' `mountArtifacts`, CL-8188). export const ArtifactDetailSchema = type({ artifact: ArtifactRowSchema, }); @@ -179,7 +177,7 @@ export function rejectApproval( } /** - * Sandboxed HTML preview URL for a Library artifact (CL-5879) — the same + * Sandboxed HTML preview URL for a Library artifact — the same * path an `