Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@towry
Copy link
Owner

@towry towry commented Oct 14, 2025

No description provided.

towry added 9 commits October 12, 2025 18:17
Remove redundant aliases and clean up package imports. Adjust delta diff tool
settings for better readability with 2-space tabs and disabled side-by-side view.

Review notes:

- No bugs found in configuration changes
- Good cleanup of unused aliases (gstat, flog)
- Delta configuration changes improve readability
- Consider if the commented git-interactive-rebase-tool package should be removed entirely
Add new rule about avoiding debug/demo code in components. Simplify and clean up
claude subagent delegation documentation by removing examples and tool preferences
while keeping essential delegation requirements.
Add Claude wrapper script for Vue.js development with specialized system prompt.
Update coding rules to include brightdata MCP tool usage for knowledge gaps.

Review notes:

- No bugs found in implementation
- Good separation of concerns with dedicated Vue prompt file
- Proxy configuration is properly inherited from main wrapper
- Consider making system prompt file path configurable for flexibility
- Vue prompt content is comprehensive and well-structured
Remove sequential-thinking tool from Oracle agent and review-change command
configurations. Delete the MCP server configuration from codex default.nix.
Switch codex MCP tool from openrouter to edenai provider and update documentation
to specify brightdata MCP usage for fetching latest context.

Review notes:

- Configuration change is properly structured with new edenai provider block
- Documentation update provides clear guidance for codex tool usage
- No apparent bugs or security issues in the configuration
- Consider validating edenai API key availability in configuration
Add bash script to generate jj diffs in temporary files and open them in Ghostty
with nvim. Update Claude settings to allow GitHub search MCP permission.

Review notes:

- Script properly handles temporary file creation with secure permissions
- Good error handling for file creation and diff generation failures
- Temporary files are cleaned up appropriately on error
- Consider adding cleanup for successful runs to prevent temp file accumulation
- Script avoids /T/ path issues as noted in comments
Add comprehensive Kiro project management system with templates for Claude AI
integration, including design documents, task lists, and initialization scripts.
Update branch naming conventions to remove towry/ prefix and modify jj-fork
function accordingly.

Review notes:

- Good separation of concerns with template files and initialization script
- Branch naming convention change is consistent across all affected files
- Consider adding validation for template variables in init script
- The TODO comment in git-town.nix should be addressed to maintain consistency
- Template files provide good structure for project management workflow
Add comprehensive Claude AI workflow scripts including boot, PR handling, and
system prompt management. Reorganize assets structure and update configuration
for improved AI development workflow.

Review notes:

- No critical bugs detected in the implementation
- Good separation of concerns between different workflow scripts
- Consider adding error handling for missing environment variables in scripts
- The quiet mode implementation in init-kiro.sh is a nice touch for CI/CD
- Recommend adding documentation for the new workflow commands
- Settings.json changes expand access appropriately for development workflow
@what-the-diff
Copy link

what-the-diff bot commented Oct 14, 2025

PR Summary

  • Changes in Program Dependencies
    Some redundant dependencies, namely html2markdown and block-goose-cli, have been commented out in the nix/darwin/apps.nix file.

  • New Configuration File for agpod
    A new settings file, nix/hm/agpod/default.nix, has been created to manage settings for the agpod tool. This deals mainly with templates and various plugin settings.

  • Enhancements in claude Configuration
    The configuration of claude, located in nix/hm/ai/claude/default.nix, now includes a directory for managing Kiro PR operations. Apart from this, the system prompt settings have also been revised to better interact with Kiro specifications.

  • Updates in codex Configuration
    The nix/hm/ai/codex/default.nix has received an update to include a new config file for experimental instructions. Plus, openai/gpt-5-codex models have been added and summary settings have been adjusted for better performance.

  • Code Cleanup in editor.nix
    Unnecessary comments on ast-grep have been removed in the nix/hm/editor.nix file.

  • Additions to Package List in frontend-dev.nix
    The package list in nix/hm/frontend-dev.nix now includes concurrently and fastmod.

  • Comment Addition in git-town.nix
    A new comment has been added to nix/hm/git-town.nix to remind about prefixing towry/ in aliases.

  • Modifications in git Settings
    The nix/hm/git.nix file now has the git-interactive-rebase-tool commented out. Several other git configuration options, like line number formatting and side-by-side view settings, have also been adjusted.

  • Shared Package Addition
    The nix/hm/shared.nix file now includes argc to the list of shared packages.

  • Agpod Inclusion in Home Configuration
    The agpod tool has now been included in the nix/home.nix file, so it is managed within the home configuration.

Remove obsolete agent configuration files (designer, kick-muse, muse) and
update model references in oracle and opencode configuration. Add new model
definitions and MCP server configurations while removing unused Brave search
integration.
@towry towry merged commit 6dd4206 into main Oct 14, 2025
1 check passed
@towry towry deleted the next branch October 14, 2025 16:03
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.

2 participants