humanspeak / svelte-markdown / 20780161588

95%

main: 95%

Build Type

push

github

Committed by web-flow

Commit Message

feat: filter utilities refactor, CI improvements, and type fixes (#231)

* Package and cleanups

* CI/CD

* Refactor filter utilities and expand test coverage

- Create generic createFilterUtilities.ts factory to eliminate ~180 lines
  of duplicate logic between unsupportedRenderers.ts and unsupportedHtmlRenderers.ts
- Replace custom MemoryCache implementation with @humanspeak/memory-cache package
- Add Escape.test.ts for previously untested component
- Expand test coverage for Code, Heading, Link, and List components with
  edge cases (special characters, empty values, boundary conditions)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* minor package cleanup

* Add tests for createFilterUtilities factory

- Test buildUnsupported: maps all keys to unsupported component
- Test allowOnly: default components, custom components, invalid keys
- Test excludeOnly: exclusions, overrides, precedence rules
- Test factory isolation: independent instances

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* ci: improve npm-publish workflow security and reliability

- Exclude log.ts from console.log checks (intentionally uses console.log)
- Remove pnpm cache from setup-node to avoid cache issues
- Add pnpm build before test to ensure fresh build
- Remove dependency caching to reduce cache-poisoning risk

* fix: resolve TypeScript type constraint errors

- Change HTMLElement[] to Element[] in test (container.children returns Element[])
- Add string constraint to HtmlKey type (keyof returns string | number)
- Use Omit<Renderers, 'html'> instead of Partial<Renderers> for proper type constraint

* ci: update workflow step names to reflect Node.js 24

Correct the step names in GitHub Actions workflows to accurately
reflect the Node.js version being used (24 instead of 22).

* ci: migrate to Blacksmith runners and add Node.js... (continued)

Run Details

195 of 223 branches covered (87.44%)

Branch coverage included in aggregate %.

40 of 40 new or added lines in 3 files covered. (100.0%)

6 existing lines in 5 files now uncovered.

433 of 440 relevant lines covered (98.41%)

21515.25 hits per line