Skip to content

feat: add split-tags strategy for grouped-by-tag (crate 0.4.0)#204

Merged
mcarvin8 merged 1 commit intomainfrom
feat/split-by-tags
Feb 11, 2026
Merged

feat: add split-tags strategy for grouped-by-tag (crate 0.4.0)#204
mcarvin8 merged 1 commit intomainfrom
feat/split-by-tags

Conversation

@mcarvin8
Copy link
Owner

  • Upgrade xml-disassembler to 0.4.0; add splitTags option to Neon bindings (parse tag:mode:field / tag:path:mode:field into DecomposeRule and pass as 9th disassemble argument).
  • Document split tags in README under grouped-by-tag: spec format, permission-set example, reassembly unchanged.
  • Add split-tags.spec.ts: disassemble HR_Admin.permissionset-meta.xml with splitTags, reassemble, and assert output matches fixture.

- Upgrade xml-disassembler to 0.4.0; add splitTags option to Neon
  bindings (parse tag:mode:field / tag:path:mode:field into
  DecomposeRule and pass as 9th disassemble argument).
- Document split tags in README under grouped-by-tag: spec format,
  permission-set example, reassembly unchanged.
- Add split-tags.spec.ts: disassemble HR_Admin.permissionset-meta.xml
  with splitTags, reassemble, and assert output matches fixture.
@qltysh
Copy link

qltysh bot commented Feb 11, 2026

Qlty

Coverage Impact

This PR will not change total coverage.

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@qltysh
Copy link

qltysh bot commented Feb 11, 2026

1 new issue

Tool Category Rule Count
qlty Structure Function with high complexity (count = 24): disassemble 1

@mcarvin8 mcarvin8 merged commit ee8f9a7 into main Feb 11, 2026
4 checks passed
@mcarvin8 mcarvin8 deleted the feat/split-by-tags branch February 11, 2026 22:37
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.

1 participant

Comments