Skip to content

Fix/v2.1.1#57

Merged
mechanik-daniel merged 6 commits intoOutburn-IL:mainfrom
mechanik-daniel:fix/v2.1.1
Feb 14, 2026
Merged

Fix/v2.1.1#57
mechanik-daniel merged 6 commits intoOutburn-IL:mainfrom
mechanik-daniel:fix/v2.1.1

Conversation

@mechanik-daniel
Copy link
Copy Markdown
Contributor

@mechanik-daniel mechanik-daniel commented Feb 14, 2026

Pull request overview

This PR updates the FLASH tokenizer/parser to correctly support URL-based slice identifiers inside [...] (e.g., extension[http://... ]) and adds regression tests for the new behavior, along with a patch version bump to v2.1.1.

Changes:

  • Update URL tokenization to stop at additional structural delimiters (notably ]) so bracketed URL slices tokenize correctly.
  • Allow url tokens as valid slice identifiers in FLASH paths.
  • Add new FLASH slice tests covering URL slices
  • Bump package version, dependencies and lockfile.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the FLASH tokenizer/parser to correctly support URL-based slice identifiers inside [...] (e.g., extension[http://... ]) and adds regression tests for the new behavior, along with a patch version bump to v2.1.1.

Changes:

  • Update URL tokenization to stop at additional structural delimiters (notably ]) so bracketed URL slices tokenize correctly.
  • Allow url tokens as valid slice identifiers in FLASH paths.
  • Add a new FLASH slice test group covering URL slices (including InstanceOf as a canonical URL) and bump package version / lockfile.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
test/test-suite/groups/flash-slices/url-slices-in-brackets.json Adds test cases validating URL slice syntax and expected FHIR output.
test/test-suite/groups/flash-slices/url-slices-in-brackets.fume New FLASH expression using a URL directly inside slice brackets.
test/test-suite/groups/flash-slices/url-slices-in-brackets-instanceof-url.fume New FLASH expression validating InstanceOf with canonical URL and URL slice usage.
src/utils/tokenizer.js Fixes URL token scanning so URLs terminate on ] (and other structural delimiters).
src/parser.js Allows url tokens to be used as valid slice identifiers in FLASH paths.
package.json Bumps package version to 2.1.1 and updates eslint-plugin-jsdoc.
package-lock.json Updates lockfile to reflect version bump and dependency resolution changes.

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

Comment thread src/utils/tokenizer.js Outdated
@mechanik-daniel mechanik-daniel merged commit bd9945c into Outburn-IL:main Feb 14, 2026
3 checks passed
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