Merged
Conversation
…nced message handling feat(chat_event): implement PushGlobalSessionNode and PushLocalSessionNode for session management refactor(commands): reorganize command modules and improve formatting fix(history): add PartialEq to message content enums for better comparison
- Layer Navigation
Feature/collapsing nodes
… functionality, added more beautiful comments, added renaming and commenting functionality to layers
…ErrorNode, Less Than or Equal Integer, PathBuf to Path, Find Item in Array, Get Element from Array, PushArray, Random Boolean Generator, To String, Get Header, and To Struct
…d 'exec_out' with 'Start' for consistency
|
Here's the code health analysis summary for commits Analysis Summary
|
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR updates documentation for several AI node types and makes backend changes to support enhanced board execution and logging. Key changes include new markdown documentation pages for various AI nodes, modifications to Tauri backend functions (e.g. execution and log querying), and corresponding UI component updates to support additional metadata and routing changes.
Reviewed Changes
Copilot reviewed 378 out of 378 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| apps/docs/src/content/docs/nodes/AI/Generative/History/Make History.md | New documentation for creating a ChatHistory object |
| apps/docs/src/content/docs/nodes/AI/Generative/History/Get System Prompt.md | New documentation for retrieving system prompts from ChatHistory |
| apps/docs/src/content/docs/nodes/AI/Generative/History/From Messages.md | New documentation describing chat history creation from messages |
| apps/docs/src/content/docs/nodes/AI/Generative/History/Clear History.md | New documentation for clearing history (duplicate pin names issue) |
| apps/docs/src/content/docs/nodes/AI/Generative/Find Model.md | New documentation for a model selection node (title inconsistency) |
| apps/docs/src/content/docs/nodes/AI/Embedding/Load Embedding Model.md | New documentation for loading embedding models |
| apps/docs/src/content/docs/nodes/AI/Embedding/Embed Query.md | New documentation for embedding a query string |
| apps/docs/src/content/docs/nodes/AI/Embedding/Embed Document.md | New documentation for embedding a document (possible type inconsistencies) |
| apps/docs/astro.config.mjs | Reordering of autogenerated docs sections |
| apps/desktop/src-tauri/src/settings.rs | Added default directory functions for logs and temporary storage |
| apps/desktop/src-tauri/src/lib.rs | Updated board execution to return log metadata and register new stores |
| apps/desktop/src-tauri/src/functions/flow/run.rs | Introduced new functions (list_runs, query_run) with extended query logic |
| apps/desktop/src-tauri/src/functions/flow/board.rs | Updated get_open_boards to include app ID for board routing |
| apps/desktop/src-tauri/src/functions.rs | Adjusted error conversions |
| apps/desktop/components/tauri-provider.tsx | Updated API calls to support new run metadata structure |
| apps/desktop/components/app-sidebar.tsx | Updated board navigation links with new route parameters |
| apps/desktop/app/store/config/logic/page.tsx | Modified board execution call to pass run metadata |
| apps/desktop/app/store/config/layout.tsx | Adjusted run invocation and run removal based on new metadata |
| Cargo.toml & apps/backend/Cargo.toml | Added dependency (bytemuck) |
Comments suppressed due to low confidence (5)
apps/docs/src/content/docs/nodes/AI/Generative/History/Clear History.md:15
- The documentation shows two pins with the same name 'History'. Consider renaming one to clarify its purpose (e.g. 'Initial History' vs. 'Cleared History').
| History | Cleared ChatHistory | Struct | History |
apps/docs/src/content/docs/nodes/AI/Generative/Find Model.md:2
- The file name is 'Find Model.md' but the title is 'Find LLM Node'. Consider updating the title or file name for consistency.
title: Find LLM Node
apps/desktop/src-tauri/src/functions/flow/run.rs:161
- Building the query string through direct string interpolation may risk SQL injection if 'node_id' is not properly sanitized. Consider using parameterized queries or escaping special characters.
if let Some(node_id) = node_id { query_string.push_str(&format!("node_id = '{}'", node_id)); }
apps/docs/src/content/docs/nodes/AI/Embedding/Embed Document.md:13
- [nitpick] The 'Query String' pin specifies a Value Type of 'Map', which is inconsistent with similar nodes (e.g. Embed Query). Verify if 'Map' is correct or if it should be 'String'.
| Query String | The string to embed | String | Map |
apps/desktop/app/store/config/layout.tsx:83
- The 'execute_run' function is being invoked with the entire runMeta object while other calls use runMeta.run_id. Ensure the API receives consistently typed parameters.
await invoke("execute_run", { id: runMeta });
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.