Skip to content

chore(packages): drop ticket references from comments (CL-8317) - #827

Merged
TheGreatAxios merged 2 commits into
mainfrom
strip-tags-packages
Sep 17, 2026
Merged

TheGreatAxios merged 2 commits into
mainfrom
strip-tags-packages

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Summary

  • Removes Linear ticket tags (CL-1234), INTR-123, and PR/issue references (#709, workbench#709) from comments, test titles, and doc strings under packages/**, per the owner ruling that git history carries ticket provenance and code should be self-documenting.
  • After removing each tag, re-read the comment: pure restatements, history narration ("used to", "retired", "cut over"), and references to deleted modules/features were trimmed or deleted; a few oversized file-header comments were cut to the 1-2 lines that still state a non-obvious constraint.
  • Test names keep their descriptive part (test("CL-6833: ...") → test("...")).
  • Zero behavior change: only comments, test titles, README/doc prose, and a couple of ticket-only fragments inside log/error strings were touched.

Test plan

  • git grep -n "CL-[0-9]\{4\}\|INTR-[0-9]\|workbench#[0-9]" -- packages is empty
  • bun run typecheck
  • bun run lint (only pre-existing, unrelated warnings in apps/sidecar)
  • bun run fmt
  • bun test smoke on the five largest touched packages (chat-ui, connections, agent-directory, settings-ui, evals+workflows) — all pass; pre-existing DOM-env failures in chat-ui/composer.test.tsx and settings-ui/granola-webhook-card.test.tsx reproduce identically on main and are unrelated to this change

@TheGreatAxios TheGreatAxios changed the title chore(packages): drop ticket references from comments chore(packages): drop ticket references from comments (CL-8317) Sep 17, 2026
@linear-code

linear-code Bot commented Sep 17, 2026

Copy link
Copy Markdown

CL-8317

@TheGreatAxios
TheGreatAxios merged commit f3a5ee1 into main Sep 17, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant