Skip to content

Conversation

@httpdss
Copy link
Owner

@httpdss httpdss commented Aug 13, 2025

This PR addresses #99 by improving clarity and consistency of logs and user-facing messages, adding a final run summary, and enhancing the MCP server get_structure_info output.

Changes

  • Standardize file operation messages across create/update/append/skip/backup/rename with emojis and clear paths.
  • Add final summary counts for actions (created, updated, appended, skipped, backed up, renamed, folders) and dry-run counts when applicable.
  • Improve get_structure_info to render folders and nested struct/with blocks cleanly rather than raw dicts.
  • Keep all existing tests passing (107/107).

Acceptance criteria

  • Output is consistent and easy to scan; final summary present.
  • get_structure_info produces clean listings.
  • No breaking changes to existing behavior, tests updated not required.

Screenshots/GIFs

  • Docs update for screenshots can be handled separately if desired.

@httpdss httpdss added enhancement New feature or request minor labels Aug 13, 2025
@codecov
Copy link

codecov bot commented Aug 13, 2025

Codecov Report

❌ Patch coverage is 75.82418% with 22 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.67%. Comparing base (a60e222) to head (cda3f4f).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
struct_module/mcp_server.py 57.89% 3 Missing and 5 partials ⚠️
struct_module/commands/generate.py 82.05% 4 Missing and 3 partials ⚠️
struct_module/file_item.py 78.78% 7 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #109      +/-   ##
==========================================
+ Coverage   66.44%   70.67%   +4.22%     
==========================================
  Files          20       20              
  Lines        1359     1422      +63     
  Branches      230      251      +21     
==========================================
+ Hits          903     1005     +102     
+ Misses        388      343      -45     
- Partials       68       74       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@httpdss
Copy link
Owner Author

httpdss commented Aug 13, 2025

Added tests to cover the new standardized messages and final summary in GenerateCommand and improved formatting in MCP get_structure_info. All tests pass locally.

  • tests/test_messages_and_summary.py: validates summary counts and append message, and checks rich rendering for folders/struct/with.
  • Entire suite: passing.

Happy to add more coverage (backup/rename strategies, dry-run summary) if desired.

… dry-run summary counters to satisfy Codecov patch
…ures to avoid AttributeError on args.diff during recursion
@httpdss httpdss merged commit 67d3eb8 into main Aug 13, 2025
5 checks passed
@httpdss httpdss deleted the feat/better-messages-99 branch August 13, 2025 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request minor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants