Skip to content

Topological sort debugging - #658

Closed
bendudson wants to merge 3 commits into
cmacmackin/topological_sort_componentsfrom
topological-sort-debugging
Closed

bendudson wants to merge 3 commits into
cmacmackin/topological_sort_componentsfrom
topological-sort-debugging

Conversation

@bendudson

Copy link
Copy Markdown
Collaborator

Purpose

Adds capabilities and instructions on how to disable and debug component ordering.

Change Summary

A setting hermes:topological_sort_components can be set to false to turn off topological sorting (printing a warning). An integer hermes:debug_component_order will print component order and permissions if greater than zero.

Validation

Unit tests check that the ordering is performed and disabled. Other unit tests check that debug messages are printed.

AI Assistance

Codex/gpt-5.4 used to plan and implement changes.

Documentation

Added a guide on debugging component order issues.

Review Notes

`hermes:topological_sort_components` is `true` by default and runs
the topological sort algorithm. Setting it to `false` disables the
topological sort.

`hermes:debug_component_order` is an integer, 0 (off) by default.
Setting to `1` prints the input and sorted component order. Setting
to `2` prints the permissions of each component.
Section in the developer manual on how to debug component ordering.
Describes how to disable sorting and how to print the component order
and their permissions.
Other boundaries have permissions `writeBoundaryFinalIfSet`.
@bendudson

Copy link
Copy Markdown
Collaborator Author

Not needed because #445 already exists.

@bendudson bendudson closed this Sep 18, 2026
@bendudson
bendudson deleted the topological-sort-debugging branch September 18, 2026 15:24
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