Skip to content

Conversation

@subtleGradient
Copy link
Member

Summary

  • record the initial WebDriver BiDi milestones in .specs/bidi
  • scaffold the @effect-native/bidi package with build/test wiring
  • implement request/response routing and event hooks with Effect-powered tests

Testing

  • nix develop --command pnpm --filter @effect-native/bidi test
  • nix develop --command pnpm lint --fix

https://chatgpt.com/codex/tasks/task_e_68d6f9e082f8832f82a4ce3673d79f64

Copilot AI review requested due to automatic review settings September 26, 2025 20:58
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR bootstraps the foundation for WebDriver BiDi support in the effect-native fork by creating the @effect-native/bidi package with essential request/response routing infrastructure.

Key changes:

  • Created the @effect-native/bidi package with full TypeScript and build configuration
  • Implemented core BiDi service with request/response correlation and event handling using Effect primitives
  • Added comprehensive test coverage for command routing and error handling scenarios

Reviewed Changes

Copilot reviewed 14 out of 15 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.specs/bidi/progress.md Documents WebDriver BiDi implementation milestones and current progress
packages-native/bidi/package.json Package configuration with Effect peer dependencies and build scripts
packages-native/bidi/src/BiDi.ts Public API exports for BiDi types and services
packages-native/bidi/src/index.ts Main package entry point
packages-native/bidi/src/internal/BiDi.ts Core BiDi service implementation with request routing and event handling
packages-native/bidi/test/BiDi.test.ts Test suite covering command correlation and error scenarios
packages-native/bidi/tsconfig.*.json TypeScript configuration files for build and test compilation
packages-native/bidi/vitest.config.ts Vitest test configuration
packages-native/bidi/README.md Package documentation with usage examples
packages-native/bidi/LICENSE License reference file
packages-native/bidi/CHANGELOG.md Package changelog initialization
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting

@subtleGradient
Copy link
Member Author

@Codes unbreak the checks. looks like pnpm docgen is failing

[21:20:54.641] ERROR (#12):
DocgenError: [Core.parseModules] The following error(s) occurred while parsing the TypeScript source files:
Missing @SInCE tag in src/internal/BiDi.ts# documentation
at /home/runner/work/effect-native/effect-native/node_modules/.pnpm/@effect+docgen@0.4.7_tsx@4.20.5_typescript@5.9.2/node_modules/@effect/docgen/bin.cjs:298136:17
at /home/runner/work/effect-native/effect-native/node_modules/.pnpm/@effect+docgen@0.4.7_tsx@4.20.5_typescript@5.9.2/node_modules/@effect/docgen/bin.cjs:272840:31
undefined
/home/runner/work/effect-native/effect-native/packages-native/bidi:
 ERR_PNPM_RECURSIVE_EXEC_FIRST_FAIL  Command failed with exit code 1: docgen

@subtleGradient subtleGradient moved this to FINISH in Native Effect Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: FINISH

Development

Successfully merging this pull request may close these issues.

1 participant