docs: add Linear integration to all user-facing and architecture docs#1147
docs: add Linear integration to all user-facing and architecture docs#1147
Conversation
nhopeatall
left a comment
There was a problem hiding this comment.
LGTM — Documentation accurately reflects the existing Linear integration implementation. All referenced code entities verified: trigger handlers (LinearCommentMentionTrigger, LinearStatusChangedTrigger, LinearReadyToProcessLabelTrigger), router adapter (LinearRouterAdapter), config provider (loadProjectConfigByLinearTeamId), credential scoping (withLinearCredentials), webhook parser (parseLinearPayload), and config schema. The getting-started config example matches the actual linearConfigSchema shape. The trigger registration code block update in 03-trigger-system.md correctly reflects the manifest-based pattern. CI all green.
One minor thing I noticed outside the diff: docs/ARCHITECTURE.md line 103 still reads "Provider clients (GitHub, Trello, JIRA) use Node.js AsyncLocalStorage…" — Linear is missing from that list. Not blocking since it wasn't part of this PR's diff, but worth a follow-up.
🕵️ claude-code · claude-opus-4-6 · run details
Summary
Linear integration is fully built but was missing from nearly all user-facing and architecture docs. This PR adds Linear alongside every Trello/JIRA mention across 9 doc files.
integration-setconfig command), update webhook note and test-it stepPOST /linear/webhookendpoint, addlinearto JOB_TYPE values andLinearJobDatato job types unionsourcetype, update registration code block to match actual manifest-based pattern, add Linear triggers tableloadProjectConfigByLinearTeamId, Linear team ID to cache description,withLinearCredentialsAsyncLocalStorage exampleLINEAR_API_KEYandLINEAR_WEBHOOK_SECRETto protected env keys listBonus fix: The trigger registration code block in
03-trigger-system.mdwas outdated (showed per-platformregisterXxxTriggers()calls); updated to match the actual manifest-based pattern (listPMProviders()iteration).No code changes — documentation only.
Card: https://trello.com/c/yWlf9XZa/610-our-top-level-readmemd-does-not-mention-our-linear-integration-weve-just-added-as-need-to-add-it-make-sure-other-docs-cover-it-t
Test plan
npm run lintpasses (confirmed ✅)🕵️ claude-code · claude-sonnet-4-6 · run details