Skip to content

Fix CI: Exclude typedoc test from regular test run - #27

Merged
miccy merged 2 commits into
merge/common-v8-04-02-2026from
copilot/sub-pr-22
Feb 5, 2026
Merged

Fix CI: Exclude typedoc test from regular test run#27
miccy merged 2 commits into
merge/common-v8-04-02-2026from
copilot/sub-pr-22

Conversation

Copilot AI commented Feb 5, 2026

Copy link
Copy Markdown

The typedoc plugin test runs during bun run test but requires generated docs that don't exist until bun run build:docs executes later in the verify script, causing ENOENT failures.

Changes

  • vitest.config.mts: Exclude scripts/typedoc-plugin-evolu.test.mts from scripts project
  • scripts/typedoc-plugin-evolu.test.mts: Document exclusion reason

The test now only runs explicitly after doc generation via bunx vitest run scripts/typedoc-plugin-evolu.test.mts at the end of the verify script.

// vitest.config.mts
{
  test: {
    name: "scripts",
    include: ["scripts/**/*.test.mts"],
    // Exclude typedoc test because it requires generated docs.
    // It runs explicitly after build:docs in the verify script.
    exclude: ["scripts/typedoc-plugin-evolu.test.mts"],
  },
}

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: miccy <9729864+miccy@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix CI issues and address review feedback on PR #22 Fix CI: Exclude typedoc test from regular test run Feb 5, 2026
Copilot AI requested a review from miccy February 5, 2026 06:07
@miccy
miccy marked this pull request as ready for review February 5, 2026 06:52
@coderabbitai

coderabbitai Bot commented Feb 5, 2026

Copy link
Copy Markdown

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

Comment @coderabbitai help to get the list of available commands and usage tips.

@miccy
miccy merged commit dcec9a6 into merge/common-v8-04-02-2026 Feb 5, 2026
1 of 2 checks passed
@miccy
miccy deleted the copilot/sub-pr-22 branch February 5, 2026 06:52
miccy added a commit that referenced this pull request Feb 5, 2026
* Clarify ESLint 'Unsafe...' troubleshooting

* Migrate to monorepo Vitest config

Introduce a root vitest.config.ts and split package test configs into unit and browser projects (packages/common/vitest.unit.config.ts, packages/common/vitest.browser.config.ts). Remove the old combined common vitest config and migrate per-package coverage/test settings into the central config. Update package.json scripts to run Vitest from the repo root, remove per-package Vitest deps where appropriate, and add browser/playwright-related devDeps at the root. Adjust tsconfigs (add root tsconfig.json and include vitest.*.config.ts in packages/common) and simplify turbo.json task entries. Also add a README link to the Vitest VS Code extension.

* Expand TDD guidance and add Vitest examples

Replace the brief Test-driven development note with an expanded TDD section that explains how to run tests, test file locations, and filtering by test name. Add subsections on test structure, type testing (expectTypeOf), and inline snapshot usage with Vitest code examples. Remove the previous Vitest filtering CLI snippets and some commit-message examples to streamline the guidance and surface more actionable testing patterns for contributors.

* Update TreeShaking.test.ts

* Use 'observability' term in error comment

* Rename vitest config to .mts and add scripts tests

Rename vitest.config.ts → vitest.config.mts (switch to .mts/ESM), update tsconfig include to reference vitest.config.mts, and add a new "scripts" test project in the Vitest config to include tests matching scripts/**/*.test.mts. Also simplify package.json "verify" script by removing the generic "pnpm test" step.

* Add createQueryBuilder, replace createQuery usage

Introduce a standalone createQueryBuilder(Schema) to build typed Kysely queries without an Evolu instance. The createQuery implementation was moved into packages/common/src/local-first/Schema.ts and exported from index.ts; Evolu.createQuery was removed from the Evolu interface/instance. Updated usages across playgrounds to call createQuery(...) (created via createQueryBuilder) and adjusted types/docs in Query.ts. Added a changeset noting the API migration and guidance to replace evolu.createQuery with createQueryBuilder(Schema).

* chore: format

* update: minors

* fix: missing dependencies and verified build

* chore: Update Biome to version 2.3.14 and reflect this change in documentation and configuration schema.

* Update packages/common/src/local-first/Evolu.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Miccy <code@miccy.dev>

* Update packages/common/src/local-first/Evolu.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Miccy <code@miccy.dev>

* Revert EvoluMinimalExample.tsx to working implementation (#24)

* Initial plan

* Revert EvoluMinimalExample.tsx to working state from 13fab62

Co-authored-by: miccy <9729864+miccy@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: miccy <9729864+miccy@users.noreply.github.com>

* Fix CI: generate docs before typedoc-plugin tests (#23)

* Initial plan

* Update all examples to use createQueryBuilder instead of evolu.createQuery

Co-authored-by: miccy <9729864+miccy@users.noreply.github.com>

* Fix CI: add build:docs before typedoc-plugin tests in verify script

Co-authored-by: miccy <9729864+miccy@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: miccy <9729864+miccy@users.noreply.github.com>

* Fix CI: Exclude typedoc test from regular test run (#27)

* fix: Correct `Uint8Array` type signature in `exportDatabase` and related callbacks.

* chore: Update Angular and Svelte dependencies and enable documentation tests in the verify script.

* refactor: improve schema difference calculation for columns and use a local constant for current schema.

---------

Signed-off-by: Miccy <code@miccy.dev>
Signed-off-by: Miccy <support@miccy.dev>
Co-authored-by: Daniel Steigerwald <daniel@steigerwald.cz>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
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