Skip to content

feat: implement MQTT message parsing and improve monitor command#56

Merged
krjordan merged 2 commits intomainfrom
15-add-basic-printer-connection-command
Jul 29, 2025
Merged

feat: implement MQTT message parsing and improve monitor command#56
krjordan merged 2 commits intomainfrom
15-add-basic-printer-connection-command

Conversation

@krjordan
Copy link
Owner

Summary

This PR implements MQTT message parsing functionality and improves the monitor command to use stored configurations.

Changes

  • Added comprehensive message parsing module for MQTT JSON messages
  • Support for and message types
  • Real-time printer status display with progress tracking and icons
  • Improved monitor command to use stored printer configurations
  • Monitor command can now run without arguments (uses default printer)
  • Added flexible parameter overrides for monitor command
  • Updated tests to properly isolate configuration using environment variables
  • Updated README and ROADMAP documentation

Test Plan

  • All unit tests pass
  • All integration tests pass (fixed timeout issues)
  • Tested with real printer - successfully receives and parses status messages
  • Monitor command works with stored config
  • Monitor command works with direct parameters
  • Clippy shows no warnings

Closes #15

- Add comprehensive message parsing module for MQTT JSON messages
- Support print.push_status and pushing.pushall message types
- Implement real-time printer status display with progress tracking
- Improve monitor command to use stored configurations
- Allow monitoring without arguments (uses default printer)
- Add flexible parameter overrides for monitor command
- Update tests to properly isolate configuration
- Update documentation with new usage patterns

Closes #15
@krjordan krjordan linked an issue Jul 29, 2025 that may be closed by this pull request
4 tasks
@krjordan krjordan self-assigned this Jul 29, 2025
@krjordan krjordan merged commit 49e933f into main Jul 29, 2025
13 checks passed
@krjordan krjordan deleted the 15-add-basic-printer-connection-command branch July 29, 2025 07:27
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.

Add basic printer connection command

1 participant