Skip to content

refactor(linear): group issue-context reads under concrete owners - #31686

Merged
nwparker merged 2 commits into
mainfrom
nwparker/module-organization-linear-issue-context
Oct 21, 2026
Merged

nwparker merged 2 commits into
mainfrom
nwparker/module-organization-linear-issue-context

Conversation

@nwparker

@nwparker nwparker commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor
Files Added Deleted Net
Test 11 $\color{#1a7f37}{\Huge{\mathbf{+}}}$​50 $\color{#cf222e}{\Huge{\mathbf{−}}}$​52 $\color{#cf222e}{\Huge{\mathbf{−}}}$​2
Prod 33 $\color{#1a7f37}{\Huge{\mathbf{+}}}$​113 $\color{#cf222e}{\Huge{\mathbf{−}}}$​149 $\color{#cf222e}{\Huge{\mathbf{−}}}$​36

Linear issue-context reads, pagination, includes, relations, and errors are spread across sibling files and forwarding exports. Group their implementations and existing tests under src/main/linear/issue-context/, with all callers and mocks updated atomically.

Remove the context entry’s four convenience exports and nine affected runtime dependency exports. Consumers import concrete implementation owners directly; moved code also imports shared result types from their defining files. Preserve existing behavior, error identity, wire formats, and historical test data.

Validation: 21 targeted suites / 178 tests; Node and CLI typechecks; changed-code quality; import boundaries; all 14 ratchets; formatting and moved-test discovery. Committed production declaration bodies remain byte-identical.

@coderabbitai

coderabbitai Bot commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 8 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 10 included reviews currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 20933458-e77d-4f3a-a272-5f5062d65fd2
📥 Commits

Reviewing files that changed from the base of the PR and between 926decd and c45d791.

📒 Files selected for processing: 44
  • src/main/linear/issue-activity-raw.ts
  • src/main/linear/issue-context/client.test.ts
  • src/main/linear/issue-context/client.ts
  • src/main/linear/issue-context/current.test.ts
  • src/main/linear/issue-context/current.ts
  • src/main/linear/issue-context/errors.test.ts
  • src/main/linear/issue-context/errors.ts
  • src/main/linear/issue-context/fanout.ts
  • src/main/linear/issue-context/includes.test.ts
  • src/main/linear/issue-context/includes.ts
  • src/main/linear/issue-context/inline-media.test.ts
  • src/main/linear/issue-context/pagination.ts
  • src/main/linear/issue-context/raw.ts
  • src/main/linear/issue-context/read-context.test.ts
  • src/main/linear/issue-context/read-context.ts
  • src/main/linear/issue-context/relations.test.ts
  • src/main/linear/issue-context/relations.ts
  • src/main/linear/issue-context/workspaces.ts
  • src/main/linear/issue-relation-mutation.ts
  • src/main/linear/issue-relation-write.ts
  • src/main/linear/issue-search-summary.test.ts
  • src/main/linear/linear-issue-lookups.ts
  • src/main/linear/mcp-issue-list-cursor.ts
  • src/main/linear/mcp-issue-list.ts
  • src/main/runtime/rpc/methods/linear-agent-access.test.ts
  • src/main/runtime/rpc/methods/linear-agent-access.ts
  • src/main/runtime/runtime-linear-command-dependencies.ts
  • src/main/runtime/runtime-linear-comment-action.ts
  • src/main/runtime/runtime-linear-comment-commands.ts
  • src/main/runtime/runtime-linear-comment-edit.test.ts
  • src/main/runtime/runtime-linear-comment-lookup-commands.ts
  • src/main/runtime/runtime-linear-connection-commands.ts
  • src/main/runtime/runtime-linear-create-commands.ts
  • src/main/runtime/runtime-linear-dedupe-commands.ts
  • src/main/runtime/runtime-linear-label-write-commands.ts
  • src/main/runtime/runtime-linear-project-write-commands.ts
  • src/main/runtime/runtime-linear-read-commands.ts
  • src/main/runtime/runtime-linear-retry-commands.ts
  • src/main/runtime/runtime-linear-save-commands.ts
  • src/main/runtime/runtime-linear-save-fields-commands.ts
  • src/main/runtime/runtime-linear-state-commands.ts
  • src/main/runtime/runtime-linear-task-fields-commands.ts
  • src/main/runtime/runtime-linear-team-write-commands.ts
  • src/main/wsl-move-incidental-writes.test.ts
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@nwparker nwparker closed this Oct 21, 2026
@nwparker nwparker reopened this Oct 21, 2026
@nwparker
nwparker merged commit 4ed095d into main Oct 21, 2026
65 of 81 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