Skip to content

feat(pm): add friction slot accessors#1294

Merged
aaight merged 1 commit into
devfrom
feature/friction-pm-slot-contract
May 9, 2026
Merged

feat(pm): add friction slot accessors#1294
aaight merged 1 commit into
devfrom
feature/friction-pm-slot-contract

Conversation

@aaight
Copy link
Copy Markdown
Collaborator

@aaight aaight commented May 9, 2026

Summary

Adds the provider-neutral PM friction slot contract requested by https://trello.com/c/69ff6a27f1ffa38b6e5f2cbc.

Changes

  • Added getFrictionContainerId(project) for Trello, JIRA, and Linear with the same guarded container semantics as the alerts slot.
  • Added getFrictionStatusDestination(project) for provider-native friction destinations.
  • Documented friction as a recognized optional Trello list key and JIRA/Linear status key in provider config schema comments.
  • Added focused unit coverage for provider behavior, missing-slot behavior, and schema acceptance.

Testing

  • npx vitest run --project unit-core tests/unit/pm/config-friction-slot.test.ts tests/unit/pm/config-alert-accessors.test.ts tests/unit/pm/config-alert-slot.test.ts
  • npm run lint:fix
  • npm run lint
  • npm run typecheck

Note: npm run lint passes with existing unrelated warnings in tests/unit/backends/postProcess.test.ts and tests/unit/gadgets/github/core/createPR.test.ts.

🕵️ codex · gpt-5.5 · run details

@codecov
Copy link
Copy Markdown

codecov Bot commented May 9, 2026

Codecov Report

❌ Patch coverage is 90.00000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/pm/config.ts 90.00% 3 Missing ⚠️

📢 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 - the new friction accessors match the work item contract and mirror the existing alerts-slot semantics across Trello, JIRA, and Linear. The optional schema contract is documented, focused coverage is present, and CI is green.

🕵️ codex · gpt-5.5 · run details

@aaight aaight merged commit baa6e66 into dev May 9, 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