Skip to content

fix(linear): wire up lookupProject to loadProjectConfigByLinearTeamId#1103

Merged
aaight merged 1 commit intodevfrom
fix/linear-lookup-project
Apr 14, 2026
Merged

fix(linear): wire up lookupProject to loadProjectConfigByLinearTeamId#1103
aaight merged 1 commit intodevfrom
fix/linear-lookup-project

Conversation

@aaight
Copy link
Copy Markdown
Collaborator

@aaight aaight commented Apr 14, 2026

Summary

  • Fixes critical bug where LinearIntegration.lookupProject() always returned null, breaking webhook processing for all Linear projects
  • Wires up the already-existing loadProjectConfigByLinearTeamId() from src/config/provider.ts — a one-line fix following the same pattern as JiraIntegration.lookupProject()
  • Updates tests/unit/pm/linear/integration.test.ts with tests covering the fixed behavior (match found and no match)

Trello card: https://trello.com/c/69deb052969af929d6fbb63f

Test plan

  • lookupProject returns correct project+config when called with a valid Linear teamId
  • lookupProject returns null when no project matches the given teamId
  • All 33 existing Linear integration tests continue to pass
  • TypeScript typecheck passes with zero errors
  • Biome lint passes with zero errors

🤖 Generated with Claude Code

🕵️ claude-code · claude-sonnet-4-6 · run details

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link
Copy Markdown
Collaborator

@nhopeatall nhopeatall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM — Clean one-line fix that wires up lookupProject() to the existing loadProjectConfigByLinearTeamId(), following the exact same pattern as JiraIntegration.lookupProject() and TrelloIntegration.lookupProject(). The ?? null coercion from undefined to null correctly satisfies the PMIntegration interface return type. Tests cover both match and no-match paths. All CI checks pass.

🕵️ claude-code · claude-opus-4-6 · run details

@aaight aaight merged commit 48eb254 into dev Apr 14, 2026
9 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.

2 participants