Skip to content

chore(hub,sidecar,tools,workflows): comments say why, never what; long ones move to docs/ (CL-8377) - #846

Merged
TheGreatAxios merged 32 commits into
mainfrom
cl-8377-apps-tools-comments
Sep 18, 2026
Merged

TheGreatAxios merged 32 commits into
mainfrom
cl-8377-apps-tools-comments

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Trims comments across apps/hub/src, apps/sidecar/src (including the full workflow-host-wiring.ts), and 22 tools/* source files so each block explains why, not what.
  • Blocks whose rationale needed more than 1-3 lines were moved to docs/, with a one-line link left in code: docs/sidecar-tool-package-durability.md and docs/sidecar-conversation-durability.md (both new).
  • No behavior, names, or types changed; unrelated lines untouched.
  • Rebased onto latest main: dropped commits that only touched now-deleted tools/{github,gotenberg,reddit,granola,linear} packages, and git rm'd files deleted upstream on every modify/delete conflict.
  • Removed a stale INTR-310 reference.

Test plan

  • find-blocks scanner confirms zero remaining 8+-line comment blocks in every file this PR touches
  • bun run check green after a full bun install refresh (typecheck, lint, fmt, full test suite)

@linear-code

linear-code Bot commented Sep 18, 2026

Copy link
Copy Markdown

CL-8377

@TheGreatAxios
TheGreatAxios force-pushed the cl-8377-apps-tools-comments branch from 4177d29 to 93792c9 Compare September 18, 2026 02:02
@TheGreatAxios
TheGreatAxios merged commit 25b8813 into main Sep 18, 2026
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