Skip to content

Feature/186 layers should indicate execution#193

Merged
felix-schultz merged 2 commits intodevfrom
feature/186-layers-should-indicate-execution
Aug 17, 2025
Merged

Feature/186 layers should indicate execution#193
felix-schultz merged 2 commits intodevfrom
feature/186-layers-should-indicate-execution

Conversation

@felix-schultz
Copy link
Copy Markdown
Member

This pull request introduces several improvements and refactors across the flow board UI components and supporting utilities. The main focus is on enhancing the layer node visualization, providing clearer execution and log status indicators, and improving code consistency and readability by standardizing imports and formatting. Additionally, there are updates to configuration and documentation to clarify usage and best practices.

Layer Node Visualization & Execution Status:

  • Enhanced the LayerNode component to display log severity icons (fatal, error, warning) and execution status indicators (running spinner, done checkmark, trace icon) using new state and hooks for log aggregation and execution tracking. The node now visually dims when execution is complete. [1] [2] [3]
  • Added logic to LayerNode for recursively collecting child node IDs and determining execution status and severity from run and log state.

Code Consistency & Formatting:

  • Standardized import formatting for React hooks and types in multiple files (layer-editing-menu.tsx, layer-inner-node.tsx, layer-node.tsx, flow-board-utils.tsx) and consistently terminated statements with semicolons. [1] [2] [3] [4] [5] [6] [7] [8]
  • Improved function signatures and type annotations for clarity and maintainability, especially for board reference and layer editing utilities. [1] [2] [3] [4]

Configuration & Documentation:

  • Updated Astro website config to fix the integrations array for plugins and improve readability.
  • Added a documentation section specifying that diff mode code snippets should not be returned, favoring drop-in replacements for easier copy-pasting.

UI Cleanups:

  • Removed commented-out legacy code and improved memoization usage in flow-node.tsx and related components for better performance and readability.

These changes collectively improve the user experience, code maintainability, and developer clarity across the flow board UI and related utilities.

@felix-schultz felix-schultz merged commit 249b611 into dev Aug 17, 2025
2 of 4 checks passed
@felix-schultz felix-schultz deleted the feature/186-layers-should-indicate-execution branch August 17, 2025 12:16
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