TEST DONT MERGE #3
Closed
Conversation
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
…ocal helper - Added a comprehensive plan for implementing dual-target support, enabling seamless integration between Electron and web applications. - Included detailed todos for reviewing and enhancing IPC coverage, path handling, and project initialization across platforms (mac/Win/Linux/WSL). - Designed a local helper service for filesystem operations, ensuring consistent behavior between environments and improved user experience. - Emphasized security measures, including localhost binding and CORS rules, while documenting platform-specific usage and helper functionality. This update lays the groundwork for a robust cross-platform application experience.
… helper service - Introduced .gitignore files for both the main application and helper service to manage dependencies and temporary files. - Added README.md for the main application, outlining features, contributing guidelines, and support resources. - Created start scripts (start-dev.sh and start.bat) for easy development environment setup. - Implemented connection testing scripts (test-connection.sh) to verify helper service connectivity. - Established a comprehensive helper service with API design documentation, including routes for filesystem operations and dialog support. This update lays the groundwork for a robust development environment and enhances the overall project structure.
Shironex
added a commit
that referenced
this pull request
Dec 20, 2025
- Create shared libs/tsconfig.base.json to eliminate duplication across 6 packages - Update all lib tsconfig.json files to extend base config - Fix build ordering to ensure sequential dependency chain (types -> utils -> platform/model-resolver/dependency-resolver -> git-utils) - Add .gitignore patterns to prevent compiled files in src directories Resolves PR review issues #3 and #4 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This was referenced Dec 20, 2025
Closed
juniorcammel
pushed a commit
to juniorcammel/automaker
that referenced
this pull request
Dec 29, 2025
Phase 1 complete - Document fixes for 3 MCP server bugs: - Bug AutoMaker-Org#1: JSON format inconsistency (array vs object, IDs lost) - Bug AutoMaker-Org#2: HTTP error handling (cryptic error messages) - Bug AutoMaker-Org#3: Race condition in auto-test (server not found) All bugs validated with multiple MCP server types (stdio + HTTP). Patches available for re-application after upstream updates. Related to upstream PR AutoMaker-Org#286 (MCP server support).
juniorcammel
pushed a commit
to juniorcammel/automaker
that referenced
this pull request
Dec 29, 2025
Fixes critical issues preventing MCP servers from working properly: Bug AutoMaker-Org#1: Add syncSuccess validation before showing success toasts - Prevents misleading success messages when save actually failed - Shows proper error toast if sync to disk fails Bug AutoMaker-Org#2: Fix MCP export to use array format with preserved IDs - Changes from object with server names as keys to array format - Preserves server IDs and names in export JSON Bug AutoMaker-Org#3: Improve HTTP error handling with better error messages - Extracts error messages from response JSON when available - Falls back to status text for better error reporting Bug AutoMaker-Org#4: Enable MCP functionality in web mode (CRITICAL) - Removes isElectron() check from syncSettingsToServer() - Allows web mode to save MCP servers to disk via backend - Without this fix, web mode cannot save any MCP configuration All changes maintain backward compatibility with Electron mode. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
4 tasks
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.