Skip to content

refactor: decompose modules by vocabulary - #227

Draft
binaryaaron wants to merge 4 commits into
mainfrom
refactor/module-vocabulary
Draft

refactor: decompose modules by vocabulary#227
binaryaaron wants to merge 4 commits into
mainfrom
refactor/module-vocabulary

Conversation

@binaryaaron

@binaryaaron binaryaaron commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • split display rendering, span handling, and payload coercion into named modules
  • separate NDD observability from adapter execution and facade support from the public interface
  • add import-linter boundary contracts and the import-linter development dependency

Why

Several modules had accumulated responsibilities with different reasons to change. Display rendering shared a module with span handling and payload coercion, NDD execution shared one with observability, and the public facade also owned output and telemetry support. This made ownership unclear, increased coupling, and made structural regressions hard to detect. The refactor gives each concern a named module and adds enforced import boundaries while preserving existing behavior.

Verification

  • make check
  • make test
  • import-linter contracts
  • symbol-location and structural gates from the migration plan
  • final vocabulary legibility probe

Follow-up

Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
@binaryaaron
binaryaaron force-pushed the refactor/module-vocabulary branch from 3115023 to 92e2281 Compare August 4, 2026 22:30
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