Skip to content

fix: align native bridge capability metadata - #69

Merged
SYMBaiEX merged 1 commit into
mainfrom
fix/cycle9-issue33-bridge-capabilities
Oct 5, 2026
Merged

SYMBaiEX merged 1 commit into
mainfrom
fix/cycle9-issue33-bridge-capabilities

Conversation

@SYMBaiEX

@SYMBaiEX SYMBaiEX commented Oct 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • Align NativeBridgeAdapter and NativeProcessBridgeAdapter capability booleans and advertised state/report formats with their configured serializer channels.
  • Keep XInput and HID gamepad reports and JSONL available when state, extensions, and profile HID payloads are all suppressed. Keep supportedCommands unchanged.
  • Document that capabilities describe OpenController SDK serialization, not support at the OS, browser, device, Steam, or game layer.

This issue was selected from the repository audit. The external vibration and motion reports cited in the Cycle 9 gate are adjacent ecosystem evidence; they do not demonstrate this OpenController metadata mismatch or its prevalence.

Validation

  • bun test packages/core/src/__tests__ — 82 passed, 0 failed.
  • bun run build — 10 tasks successful.
  • bun run typecheck — passed after the workspace build generated project-reference declarations.
  • bunx @biomejs/biome check docs/adapters.md packages/core/src/adapters/native-bridge.ts packages/core/src/adapters/native-process.ts packages/core/src/__tests__/native-bridge-capabilities.test.ts — passed.
  • git diff --check — passed.

Closes #33

Copilot AI balanced review requested due to automatic review settings October 5, 2026 01:53

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@coderabbitai

coderabbitai Bot commented Oct 5, 2026

Copy link
Copy Markdown

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 49 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: c1b14fd1-bcc9-4363-88bc-b197e6b59e26
📥 Commits

Reviewing files that changed from the base of the PR and between aa57f17 and 771b37b.

📒 Files selected for processing (4)
  • docs/adapters.md
  • packages/core/src/__tests__/native-bridge-capabilities.test.ts
  • packages/core/src/adapters/native-bridge.ts
  • packages/core/src/adapters/native-process.ts
  • Autopilot · 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.

@SYMBaiEX
SYMBaiEX merged commit 42d7a3a into main Oct 5, 2026
2 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.

Align native adapter capabilities with configured bridge outputs

3 participants