Sync with base by LeeGathercole · Pull Request #5 · NHSDigital/nhso-eventcatalog

LeeGathercole · 2025-12-10T10:39:18Z

Motivation

(Write your motivation here.)

…2131) * fix(visualiser): improve icon and label spacing in nodes Add conditional bottom margins to node icons (mb-2 in full mode, mb-1 in simple mode) and increase label margins from mb-4 to mb-6 for longer labels (Channel, Command, Service, Data Product) to prevent overlap with rotated text in full mode. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> * chore: setup language-server and playground packages for beta release - Rename @eventcatalog/ec-language-server to @eventcatalog/language-server - Add language-server and visualiser to CI/CD packaging workflow - Configure changesets to ignore dsl-playground (private package) - Create beta changeset for language-server 0.1.0 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> * fix(language-server): add vitest config to resolve .js to .ts imports The Langium-generated module.ts imports grammar.js (ESM convention), but vitest needs explicit extension resolution to find the .ts source. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(language-server): use vite plugin to resolve .js to .ts imports Replace naive resolve.extensions with a proper Vite plugin that rewrites .js imports to .ts for all files in the language-server package. Handles all Langium-generated ESM imports without needing explicit aliases. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: add changeset for visualiser node redesign Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(playground): remove build scripts from private package The playground is private and doesn't need to build in CI. Removing build:bin prevents Turbo from trying to build it during test:ci, which was failing because the visualiser dist/styles.css wasn't available yet. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(sdk): use separate test fixture dir for resources test resources.test.ts and users.test.ts both used catalog-users/ as their test fixture directory, causing a race condition when vitest runs them in parallel - one test's beforeEach would delete the other's fixtures. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(language-server): resolve .js to .ts paths relative to importer The plugin was returning relative paths which Vite resolved from root / instead of from the importing file. Use path.resolve with the importer's directory to produce correct absolute paths. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Create fuzzy-ghosts-chew.md * Create proud-crabs-know.md * fix(visualiser): restore Tailwind utilities in built CSS Re-add tailwindcss/utilities import that was removed in 7d0203c. The removal stripped all utility classes (flex, rounded, shadow, etc.) from the built CSS, leaving visualiser nodes unstyled in core. To prevent the original .hidden class conflict with the Astro header, a PostCSS plugin now strips the bare .hidden rule from output and the DownloadButton uses a custom ec-download-btn class instead. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * feat(visualiser, playground, language-server): node redesign, DSL examples, and completion improvements Redesign visualiser node components with improved icon/label spacing, consistent padding, and better visual hierarchy. Add DSL example templates to playground and enhance language-server completions. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(language-server): resolve visualizer refs by version and use versioned output paths - Key topLevelDefs by name@version for exact version lookups in graph builder - Use versioned paths (name/versioned/x.y.z/index.md) in compiler to prevent multi-version collisions - Remove invalid CommonJS require export from ESM-only package Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(language-server): resolve bare refs to latest version using semver Per the DSL spec, unversioned references resolve to the latest version. Previously, resolveNodeId and topLevelDefs used insertion order which could bind bare refs to older versions. Now uses semver.compare to deterministically pick the highest version. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(language-server): infer flow ref types from nested domain definitions inferTypeFromCatalog now uses topLevelDefs (which includes nested services/containers/channels inside domains) instead of scanning only program.definitions. Previously, flow refs to nested resources would fall back to "step" type instead of the correct type. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(language-server): preserve declared message type for data-product edges Data-product inputs/outputs have an explicit type (event/command/query) in the DSL, but fallback node creation was hardcoding "event". Now uses inp.type and out.type so command/query inputs/outputs get the correct node type in the graph. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(language-server): add routes-to edges for all receive channels The dedupe check for message→channel routes-to edges was checking if the message had any routes-to edge, causing only the first channel in a multi-channel receives clause to get an edge. Now checks against the specific channel ID so all channels get their routes-to edges. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(language-server): avoid langium generate race in CI build The `build` script ran `langium generate && tsc` redundantly since `build:bin` (which runs first via turbo) already does the same. When turbo ran both `build` and `test:ci` concurrently, the second `langium generate` overwrote src/generated files while vitest was loading them, causing ERR_MODULE_NOT_FOUND for grammar.ts. Changed `build` to just `tsc` since `build:bin` already handles code generation. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>

* perf(ui): defer search/sidebar hydration on initial load * Create brown-mirrors-teach.md --------- Co-authored-by: Thor <thor@Thors-MacBook-Pro.local> Co-authored-by: David Boyne <boyneyy123@gmail.com>

* perf(search): load search index on first open * Create good-laws-burn.md --------- Co-authored-by: Thor <thor@Thors-MacBook-Pro.local> Co-authored-by: David Boyne <boyneyy123@gmail.com>

* perf(search): compact search index payload for faster transfers * Create serious-years-sin.md --------- Co-authored-by: Thor <thor@Thors-MacBook-Pro.local> Co-authored-by: David Boyne <boyneyy123@gmail.com>

* fix(core): resolve astro check type errors - Unify CatalogConfig with canonical Config type instead of maintaining a separate incomplete definition missing logo, landingPage, etc. - Add missing allowAnyEnvInSpecHeaders to Config asyncAPI field - Fix SearchModal favorite fallback object to include summary property - Fix SearchDataLoader window.setTimeout narrowing to never Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Create many-actors-jump.md * chore: disable Vercel preview deployments Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

…erns (#2152) Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>

* fix(ui): prevent theme toggle hydration mismatch * Create sharp-coins-promise.md --------- Co-authored-by: Thor <thor@Thors-MacBook-Pro.local> Co-authored-by: David Boyne <boyneyy123@gmail.com>

* feat(core): inject full catalog api route only when enabled * Create tall-donkeys-rule.md --------- Co-authored-by: Thor <thor@Thors-MacBook-Pro.local> Co-authored-by: David Boyne <boyneyy123@gmail.com>

Co-authored-by: Thor <thor@Thors-MacBook-Pro.local>

* perf(visualiser): skip visualiser route generation when disabled * Create witty-vans-cry.md --------- Co-authored-by: Thor <thor@Thors-MacBook-Pro.local> Co-authored-by: David Boyne <boyneyy123@gmail.com>

* perf(dev): prewarm first request after astro startup * Create brave-falcons-march.md --------- Co-authored-by: Thor <thor@Thors-MacBook-Pro.local> Co-authored-by: David Boyne <boyneyy123@gmail.com>

* fix(visualiser): make focus mode modal self-styled * Create clever-lions-occur.md --------- Co-authored-by: Thor <thor@Thors-MacBook-Pro.local> Co-authored-by: David Boyne <boyneyy123@gmail.com>

* updated asto versions * Create afraid-mirrors-enjoy.md