Snapshot/signal value format registry

[desmonddak]

Description & Motivation

This is a dependency needed as central for widgets to communicate changes in the way a signal value is displayed.
It replaces PR #697 which is now a draft.

Related Issue(s)

Testing

This feature is used in future widgets where testing is performed, but not introduced yet. But basic testing for store/recall of format is performed.

Backwards-compatibility

Is this a breaking change that will not be backwards-compatible? If yes, how so?

No

Documentation

Does the change require any updates to documentation? If so, where? Are they included?

Not yet.

[mkorbel1]

looks good to me!

[Copilot]

Pull request overview

Introduces shared signal-value formatting and occurrence-address storage, replacing draft PR #697.

Changes:

• Adds signal format conversion, preference storage, and notifications.
• Adds a generic occurrence-address trie.
• Adds tests, exports, dependencies, and README documentation.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
signal_value_format_registry.dart	Implements formatting and preference registry.
signal_value_format_registry_test.dart	Tests formatting and registry lookup.
rohd_devtools_widgets.dart	Exports the registry API.
rohd_devtools_widgets/pubspec.yaml	Adds hierarchy dependency.
rohd_devtools_widgets/README.md	Documents shared utilities.
occurrence_trie.dart	Implements occurrence-address trie storage.
occurrence_trie_test.dart	Tests trie operations and validation.
hierarchy_models.dart	Exports the trie API.

Suppressed comments (5)

rohd_devtools_extension/packages/rohd_devtools_widgets/README.md:47

• ExportToast does not exist in the public API; this utility is exported as showExportToast. Documenting the nonexistent type makes the usage guidance unusable.

- **`ExportToast`** — Toast notification widget for export feedback and status messages.

rohd_devtools_extension/packages/rohd_devtools_widgets/README.md:53

• There is no exported CrossProbeMenu symbol. Cross-probe menu integration is provided by top-level helpers such as buildGotoSourceMenuItems, so the README currently points users to an API they cannot import.

- **`CrossProbeMenu`** — Shared context menu integration for cross-probing actions across different ROHD DevTools surfaces.

rohd_devtools_extension/packages/rohd_devtools_widgets/README.md:57

• LogicTypeUtils is not declared or exported. The public entry point is the top-level expandLogicType function (with TypeFieldNode as its result), so this API reference is inaccurate.

- **`LogicTypeUtils`** — Utilities for working with ROHD logic types and formatting logic values for display.

rohd_devtools_extension/packages/rohd_devtools_widgets/README.md:61

• BitExpansionMenu is not a public declaration. The package exports BitExpansionMenuValues and menu-building/resolution functions instead, so consumers cannot use the documented symbol.

- **`BitExpansionMenu`** — Shared popup menu items for "Expand Bits" and "Define Bit Fields" actions used across signal selection overlays and panels.

rohd_devtools_extension/packages/rohd_devtools_widgets/README.md:69

• No RohdExtensionStatus declaration is exported. Extension status is represented by RohdModuleInfo and its RohdFormatInfo entries, so this documentation names an unusable API.

- **`RohdExtensionStatus`** — Status information and connection state for the ROHD extension.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.