Fix/skills 3#103
Merged
Merged
Conversation
…te state handling - Replaced Redux-based skill state management with hooks for improved performance, utilizing `useAvailableSkills`, `useSkillSnapshot`, and `useAllSkillSnapshots`. - Streamlined skill list derivation and sorting logic to enhance clarity and maintainability. - Updated components to reflect the new state management approach, ensuring real-time updates and compatibility with existing code. - Bumped OpenHuman version to 0.49.24 in Cargo.lock.
…encies - Simplified skill state management by replacing Redux-based logic with direct references to runtime maps. - Adjusted the SkillsGrid component to derive skill sync summary text without relying on skill states. - Removed unused Redux configurations and tests related to skills, streamlining the codebase.
notifyOAuthComplete and triggerSync no longer throw when the frontend SkillManager has no local runtime instance. They persist setup_complete via RPC first, then try core RPC pass-through as fallback. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Consolidated import statements in the SkillsGrid component for better readability. - Updated the DEV_FORCE_ONBOARDING constant in the config file to enhance clarity and maintainability by combining conditions into a single line.
- Introduced SkillDebugModal component to inspect a skill's runtime state, including metadata, published state, and tool definitions. - Integrated the modal into the SkillCard component, allowing users to open it for debugging purposes. - Implemented functionality for calling tools and displaying results, enhancing the debugging experience for skills.
- Improved the OAuth deep link process by adding steps to persist setup completion, start the skill in the core runtime, and notify the skill of OAuth completion. - Enhanced error handling for starting skills and notifying OAuth completion, ensuring resilience in the skill management workflow. - Updated logging throughout the process to provide better insights into the state and actions taken during OAuth handling and tool calls.
- Disabled the creation of updater artifacts in the Tauri configuration to streamline the build process. - Introduced a new script for building and code-signing macOS Tauri releases, including notarization steps and environment variable validation. - Updated environment loading logic to source from the correct secrets file for improved configuration management.
- Implemented pre-signing of sidecar binaries with hardened runtime and entitlements to comply with Apple notarization requirements. - Enhanced the build script to verify and sign all executables in the specified sidecar directory, ensuring proper code-signing before the build process.
…d script - Added functionality to pre-sign sidecar binaries with hardened runtime and entitlements to meet Apple notarization requirements. - Enhanced the build script to verify and sign all executables in the specified sidecar directory, ensuring compliance before the build process.
…r re-signing - Added a new step to the release workflow for re-signing sidecar binaries with hardened runtime and notarization after the build process. - Updated the build script to remove pre-signing of sidecar binaries, ensuring notarization is handled separately for compliance with Apple requirements. - Improved the overall build process by verifying and re-signing all executables within the .app bundle, including frameworks and resources, before notarization. - Implemented DMG re-packaging after notarization to ensure the latest signed application is included in the distribution.
…indowBounds - Combined separate import statements for AppContext and WindowBounds into a single line for improved readability and organization in the capture module.
- Updated log statements in various files to use multi-line formatting for improved clarity and consistency. - Enhanced the SkillDebugModal and event loop logging to streamline the output and make it easier to read. - Refactored log messages in js_handlers and ops_net to maintain uniformity in logging style.
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
1 participant
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.
Summary
Problem
Solution
Submission Checklist
app/) and/orcargo test(core) for logic you add or changeapp/test/e2e, mock backend,tests/json_rpc_e2e.rsas appropriate)//////!(Rust), JSDoc or brief file/module headers (TS) on public APIs and non-obvious modules(Any feature related checklist can go in here)
Impact
Related