Skip to content

Gen 2026 01 20#2

Open
trillium wants to merge 76 commits into
mainfrom
gen_2026_01_20
Open

Gen 2026 01 20#2
trillium wants to merge 76 commits into
mainfrom
gen_2026_01_20

Conversation

@trillium

Copy link
Copy Markdown
Owner

No description provided.

SimeonC and others added 30 commits January 20, 2026 15:48
wip: Delete workspace.json for cherry-pick
- git cherry pick 070c870
- Delete cursorless-nx/workspace.json
Add the package and initial working

There is an issue here that the jest parser requires no file name extension in imports, but ts-node requires `.js` extension. Probably will need to change this to build then generate rather than ts-node (or spend ages messing with the configs)

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

All working with tests

Need to update to be in the new NX repo format and correctly handle the newer formats for test fixtures which have changed since this was written
Attempting to resolve import error:
  Server Error
  Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
trillium and others added 24 commits January 20, 2026 16:01
-Displays before/after states
-wip: During state needs work

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 reviewed 72 out of 74 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

export * from "./generate-examples";
export * from "./loadFixture";
export * from "./test-case-component";
export * from "./components/VisualizerWrapper";

Copilot AI Jan 21, 2026

Copy link

Choose a reason for hiding this comment

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

The export for VisualizerWrapper is being added to the package index, but this component doesn't exist yet in the codebase. This will cause a module resolution error when trying to use this package.

Either remove this export until the component is implemented, or ensure the component file is created in this same PR.

Copilot uses AI. Check for mistakes.
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.

3 participants