Skip to content

test: add Writer 0.9.0 coverage gaps - #6

Open
Digidai wants to merge 1 commit into
mainfrom
cursor/writer-090-tests-coverage-7686
Open

test: add Writer 0.9.0 coverage gaps#6
Digidai wants to merge 1 commit into
mainfrom
cursor/writer-090-tests-coverage-7686

Conversation

@Digidai

@Digidai Digidai commented Aug 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • add focused node:test coverage for archive restore/reopen edge cases in src/archive-actions.js
  • add helper-level coverage for semantic, search, reader escaping, and cron janitor behavior
  • extend persist/export tests for formatting guardrails and 413 row-cap handling
  • bump version metadata to 0.9.0 (package.json, package-lock.json, src/version.js, CHANGELOG.md)

Test Areas Added

  • archive-actions restore vector lifecycle + reopen 409 guards
  • semantic helpers: feature gating, payload vector extraction, delete fallback, legacy D1 backfill fallback
  • search helpers: escapeLike, ordered hydration, strict mode parsing, no-key semantic request behavior
  • html reader output escaping for metadata + http(s)-only markdown links
  • cron janitor behavior when idle draft archive is disabled + launchPipeline guard no-op
  • persist formatting safeguards (agentFormatting === false, short formatted output fallback)
  • export 413 when archived rows exceed the 200-document cap

Validation

  • npm test
  • npm run check
Open in Web Open in Cursor 

Co-authored-by: Gene Dai <Digidai@users.noreply.github.com>
@Digidai
Digidai marked this pull request as ready for review August 16, 2026 16:24
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

cursor Bot pushed a commit that referenced this pull request Aug 16, 2026
Co-authored-by: Gene Dai <Digidai@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