-
Notifications
You must be signed in to change notification settings - Fork 5
Test coverage #16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
jamesbrink
wants to merge
35
commits into
main
Choose a base branch
from
develop
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Test coverage #16
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
- Add workflow_dispatch trigger with deploy_website input to CI pipeline - Create deploy command in web shell with AWS profile and env var support - Add comprehensive error handling and credential validation 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Change 'project' to 'Creation' in footer text - Improve branding consistency with Utensils company style 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Update string context test to use get_option instead of search_options - Fix return format of options_by_prefix tests to use array instead of dict - Update stats tests to use correct field names (total_options, by_category) - Add explicit None checks to satisfy type checker - Fix module patching in test_init.py with proper setUp/tearDown - Remove unused imports to fix linting issues 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Add tests for wildcard search behavior in search tools - Add tests for context loading states and error handling - Add cross-platform cache compatibility tests - Improve HTML parsing robustness in integration tests - Fix multi-word query handling test 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Add wildcard search tests with various search patterns - Add error handling tests for Home Manager context - Add cross-platform compatibility tests for Windows - Add multi-platform integration tests - Add test coverage for Darwin client 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Remove unused imports and variables in test files to fix flake8 linting errors: - Remove unused imports like re, uuid, sys, asyncio, logging, importlib - Fix unused variable warnings by replacing variable assignments with direct calls - Keep implementation code untouched as per requirements 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Fixed attribute access errors in HomeManagerClient and DarwinClient tests - Added proper null safety checks in integration tests - Corrected indentation issues in test files - Fixed async coroutine warnings by proper mocking - Updated dictionary access patterns for type safety 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Code Complexity Analysis
For more details, check the complexity-report artifact in the workflow run. |
❌ 15 Tests Failed:
View the top 3 failed test(s) by shortest run time
To view more test analytics, go to the Test Analytics Dashboard |
…ssues - Replace pipe-separated function names with flex wrap layout for better mobile display - Add break-all to prevent long text from causing horizontal overflow - Improve table responsiveness with smaller padding and font sizes on mobile - Maintain consistent styling across all sections following design system - Wrap table in additional container for better width control
- Change 'Add to your Claude Code configuration file' to 'Add to your MCP configuration file' - Change 'Then start asking Claude about NixOS packages and configuration options!' to 'Start leveraging NixOS package information and configuration options in your workflow!' - Make language more inclusive of different platforms that use MCP configurations
- Added real sample responses for multiple MCP-NixOS tools: - home_manager_options_by_prefix - home_manager_stats - darwin_info - darwin_list_options - darwin_options_by_prefix - darwin_stats - Improved CodeBlock component styling with consistent headers - Fixed HTML entity rendering in code examples
- Added a new Overview section above the API Reference - Created a grid layout with four informative cards: - NixOS Tools overview - Home Manager Tools overview - nix-darwin Tools overview - Why Use MCP-NixOS benefits section - Fixed lint issues with proper HTML entity escaping - Improved readability with consistent styling and formatting
- Replaced grid layout with a more visually appealing hierarchical design - Added gradient background and improved spacing for better readability - Incorporated icons for each tool category for visual distinction - Added hover effects and interactive elements for better engagement - Organized tools in collapsible sections with clear visual hierarchy - Used NixOS color palette consistently throughout the design - Improved mobile responsiveness with better column layouts
…le responses - Replaced all AnchorHeading components with regular h5 tags for example sections - Maintained SVG icons and styling for visual consistency - Added detailed sample responses for home_manager_search and home_manager_info tools - Improved documentation with realistic tool output examples - Ensured consistent styling across all documentation sections
- Added information about MCP-NixOS being a stdio implementation using FastMCP - Added a link to the FastMCP GitHub repository for reference - Created a new feature card highlighting the FastMCP implementation - Reorganized the feature cards to accommodate the new information - Improved the overview section with more technical context
- Added comprehensive JSON response format section at the top of API Reference - Clarified that Home Manager and nix-darwin data are parsed from HTML docs - Added example JSON structure with explanation of key fields - Included note about documentation examples showing only result content
- Move Response Format section to top of API Reference - Rename 'How to Use with AI Assistants' to 'Basic Chat Examples with AI Assistants' - Improve explanation of usage examples to clarify AI tool access - Fix HTML structure and nesting issues - Properly escape HTML entities
…ion sections and simplifying overview
…ing instructions - Add documentation for CollapsibleSection component and client-side React usage - Include TypeScript checking commands: npx tsc --noEmit - Add Prettier verification commands: npx prettier --check/--write - Document manual development site deployment process - Synchronize changes to all rule files 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Reorganized environment variables into Essential, Advanced, and Development categories - Added default values for all environment variables - Added configuration section anchor ID for direct navigation - Added cross-links between pages to the configuration documentation - Clarified that ELASTICSEARCH_URL is optional for most users - Fixed import for Link component in usage page
…locks, standardize header styles, fix apostrophe encoding in Nix code blocks
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.