feat: add Adrapid plugin by Vishuzz · Pull Request #662 · corsairdev/corsair

Vishuzz · 2026-08-10T08:55:29Z

Description

The youcom plugin is already implemented under packages/youcom/, but it was missing from the core provider registry.

This PR registers youcom in packages/corsair/core/constants.ts so it is recognized by the core provider system.

Changes

Added youcom to BaseProviders, alphabetically between xquik and youtube
Added youcom: 'You.com' to ProviderDisplayNames
Added youcom to the AllProviders union type
Updated packages/youcom/jest.config.cjs with the required module mappings

Verification

pnpm run validate:plugins — passed
pnpm typecheck — passed
pnpm lint — passed
pnpm --filter @corsair-dev/youcom test — 22/22 tests passed

Summary by CodeRabbit

New Features
- Added You.com as a supported provider, displayed as “You.com.”
Tests
- Improved handling of tests that require PostgreSQL or Slack connectivity.
- Strengthened timeout testing to provide more reliable validation.
Chores
- Updated You.com test configuration to resolve shared Corsair modules correctly.

vercel · 2026-08-10T08:55:36Z

@Vishuzz is attempting to deploy a commit to the corsair Team on Vercel.

A member of the Team first needs to authorize it.

coderabbitai · 2026-08-10T08:56:28Z

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: b187d6a2-379d-4b39-808e-8ec462fdab9d

📥 Commits

Reviewing files that changed from the base of the PR and between 99ade55 and 01af552.

⛔ Files ignored due to path filters (1)

pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

📒 Files selected for processing (4)

packages/corsair/core/constants.ts
packages/corsair/tests/postgres-js-database.test.ts
packages/corsair/tests/probe.test.ts
packages/youcom/jest.config.cjs

📝 Walkthrough

Walkthrough

The PR registers the You.com provider, adds Jest mappings for shared Corsair modules, improves PostgreSQL test gating when connectivity is unavailable, and makes probe timeout continuation testing deterministic.

Changes

Provider registration and test reliability

Layer / File(s)	Summary
You.com provider registration `packages/corsair/core/constants.ts`, `packages/youcom/jest.config.cjs`	The provider registries include `youcom` with the display name `You.com`. Jest maps `corsair/core` and `corsair/hub` to TypeScript sources.
Database test availability guards `packages/corsair/tests/postgres-js-database.test.ts`	The `gated` and `runIf` helpers skip tests when PostgreSQL is unavailable and preserve callback arguments and timeout handling.
Deterministic timeout continuation test `packages/corsair/tests/probe.test.ts`	The test resolves the delayed read explicitly after the probe timeout to trigger the detached continuation.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

corsairdev/corsair#330: Registers the You.com provider and updates its Jest configuration.
corsairdev/corsair#623: Adds the same You.com provider registrations.
corsairdev/corsair#653: Overlaps on the You.com provider registrations and Jest configuration.

Suggested reviewers: devjain32

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests

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

Warning

⚠️ This pull request shows signs of AI-generated slop (description_diff_mismatch). It has been flagged by CodeRabbit slop detection and should be reviewed carefully.

greptile-apps · 2026-08-10T08:58:32Z

Greptile Summary

This PR registers the existing You.com plugin in the core provider vocabulary and adds Jest aliases needed by its tests. It also includes unrelated changes to two Corsair core test files that violate the repository's plugin-PR scope boundary.

Adds youcom to BaseProviders, ProviderDisplayNames, and AllProviders.
Maps corsair/core and corsair/hub in the You.com Jest configuration.
Alters PostgreSQL test gating and probe continuation timing in core tests.

Confidence Score: 2/5

This PR should not merge until the unrelated Corsair core test changes are removed or split into a separate pull request.

The You.com registration and Jest mapping are internally consistent, but the changeset violates the repository's mandatory plugin scope boundary by modifying two core test files.

Files Needing Attention: packages/corsair/tests/postgres-js-database.test.ts, packages/corsair/tests/probe.test.ts

Important Files Changed

Filename	Overview
packages/corsair/core/constants.ts	Consistently registers youcom in the provider list, display-name map, and provider union.
packages/youcom/jest.config.cjs	Adds core and hub aliases matching the established Jest configuration used by sibling plugins.
packages/corsair/tests/postgres-js-database.test.ts	Introduces unrelated test-gating changes outside the allowed plugin PR scope and uses undocumented any types.
packages/corsair/tests/probe.test.ts	Changes an unrelated core probe test outside the allowed plugin PR scope.

_{Reviews (1): Last reviewed commit: "feat: add Adrapid plugin" | Re-trigger Greptile}

greptile-apps · 2026-08-10T08:58:35Z

+	const gated = () => {
+		const runner = liveDisabled ? it.skip : it;
+		return (name: string, fn: any, timeout?: number) => {
+			runner(
+				name,
+				async (...args: any[]) => {
+					if (!connectable) return;
+					return fn(...args);
+				},
+				timeout,
+			);
+		};
+	};


Plugin scope boundary is broken

This You.com plugin PR changes PostgreSQL test registration here and probe behavior in packages/corsair/tests/probe.test.ts, outside the permitted plugin directory, provider registry, and lockfile scope. These unrelated core test changes cause the repository's plugin-PR scope gate to fail and need to be removed or split into a separate PR.

Rule Used: A plugin PR must only modify files inside a single... (source)

Knowledge Base Used: The provider-plugin package pattern

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

github-actions · 2026-08-10T08:58:51Z

Plugin PR scorecard — `packages/youcom`

Check	Status	Notes
R1 — Scope: plugin files only	❌	Out of scope: packages/corsair/tests/postgres-js-database.test.ts, packages/corsair/tests/probe.test.ts
R2 — Tests with assertions	✅
R3 — Description complete	✅
R3 — Linked issue / claim	⚠️	No "Fixes #…" or claim link — add one if this PR has a claim or issue
R4 — Demo video / recording	❌	Required in "Screenshots / Demos" before a maintainer reviews

Rules: PLUGIN_PR_RULES.md · re-runs on every push

github-actions · 2026-08-10T08:58:54Z

Hey @Vishuzz, thanks for the contribution! 🏴‍☠️ Before a maintainer reviews, please fix the items below — the review re-runs automatically on your next push.

Must fix

P0 packages/corsair/tests/postgres-js-database.test.ts:173 — Plugin scope boundary is broken
This You.com plugin PR changes PostgreSQL test registration here and probe behavior in packages/corsair/tests/probe.test.ts, outside the permitted plugin directory, provider registry, and lockfile scope. These unrelated core test changes cause the repository's plugin-PR scope gate to fail and need to be removed or split into a separate PR.

Rule Used: A plugin PR must only modify files inside a single... (source)

Knowledge Base Used: The provider-plugin package pattern

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

PR requirements (rules)

R1 — Out of scope: packages/corsair/tests/postgres-js-database.test.ts, packages/corsair/tests/probe.test.ts
R4 — Required in "Screenshots / Demos" before a maintainer reviews

If anything remains after your next push, a bot commit will clean it up; a maintainer always does the final review and merge.

Vishuzz added 2 commits August 10, 2026 12:51

feat: add Adrapid plugin

0eb1671

feat: add Adrapid plugin

01af552

github-actions Bot added the core Changes in packages/corsair label Aug 10, 2026

Vishuzz closed this Aug 10, 2026

Vishuzz reopened this Aug 10, 2026

Vishuzz closed this Aug 10, 2026

greptile-apps Bot reviewed Aug 10, 2026

View reviewed changes

github-actions Bot added the gate:failed Plugin PR gate checks failing label Aug 10, 2026

github-actions Bot added the bot:round-1 Review bot posted consolidated findings label Aug 10, 2026

Navigation Menu

Uh oh!

feat: add Adrapid plugin - #662

feat: add Adrapid plugin#662
Vishuzz wants to merge 2 commits into
corsairdev:mainfrom
Vishuzz:feat/adrapid-plugin

Vishuzz commented Aug 10, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

vercel Bot commented Aug 10, 2026

Uh oh!

coderabbitai Bot commented Aug 10, 2026 •

edited

Loading

Review failed

Walkthrough

Changes

Uh oh!

greptile-apps Bot commented Aug 10, 2026

Uh oh!

greptile-apps Bot Aug 10, 2026

Uh oh!

github-actions Bot commented Aug 10, 2026

Uh oh!

github-actions Bot commented Aug 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

Vishuzz commented Aug 10, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Changes

Verification

Summary by CodeRabbit

Uh oh!

vercel Bot commented Aug 10, 2026

Uh oh!

coderabbitai Bot commented Aug 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review failed

Walkthrough

Changes

Uh oh!

greptile-apps Bot commented Aug 10, 2026

Greptile Summary

Confidence Score: 2/5

Important Files Changed

Uh oh!

greptile-apps Bot Aug 10, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Aug 10, 2026

Plugin PR scorecard — packages/youcom

Uh oh!

github-actions Bot commented Aug 10, 2026

Must fix

PR requirements (rules)

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Vishuzz commented Aug 10, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Aug 10, 2026 •

edited

Loading

Plugin PR scorecard — `packages/youcom`