Skip to content

fix: simplify logger, fix prompts, and restore clean interactive release (refs #67 #71 #72)#86

Merged
fxstein merged 2 commits intodevelopfrom
fix/cleanup-debug-output-release-scripts
Jun 30, 2025
Merged

fix: simplify logger, fix prompts, and restore clean interactive release (refs #67 #71 #72)#86
fxstein merged 2 commits intodevelopfrom
fix/cleanup-debug-output-release-scripts

Conversation

@fxstein
Copy link
Owner

@fxstein fxstein commented Jun 30, 2025

  • Replace overengineered logger with simple, robust version
  • Restore timestamp and branch formatting to logger output
  • Make debug output conditional on DEBUG=1 or DEBUG=true
  • Replace all echo DEBUG statements with log_debug calls
  • Remove all debug mode wrappers - logger handles debug control
  • Fix safe-prompt to output prompts to stderr (prevents command substitution issues)
  • Ensure interactive release script works without debug interference
  • Confirm all remaining echo statements are for UI or prompt output only
  • Follow Unix best practices for prompt output

Closes #67, #71, #72

fxstein added 2 commits July 1, 2025 00:17
- Replace complex over-engineered logger with simple, reliable version
- Make debug output conditional on DEBUG=1 or DEBUG=true
- Replace all echo DEBUG statements with log_debug calls
- Remove all debug mode wrappers - logger handles debug control
- Fix safe-prompt to output prompts to stderr (prevents command substitution issues)
- Ensure interactive release script works without debug interference
- Follow Unix best practices for prompt output
…ctions (refs #67 #71 #72)

- Restore timestamp and branch formatting to logger output
- Replace initial echo in release script with log_info
- Confirm all remaining echo statements are for UI or prompt output only
- Maintain simple, reliable logger structure
@github-actions
Copy link

🧪 Test Results

All tests passed

Test Suites Executed:

  • Unit Tests:
    • Logger Tests
    • Firmware Summary Tests
  • Integration Tests:
    • Configuration Tests
    • Parameter Processing Tests
    • Storage Validation Tests
    • Integration Tests

📊 Test Reports: Available in workflow artifacts
🔍 Test Logs: Available in workflow artifacts


Generated by GoProX Comprehensive Testing Framework

@fxstein fxstein merged commit dcfd2bd into develop Jun 30, 2025
7 checks passed
@github-actions github-actions bot deleted the fix/cleanup-debug-output-release-scripts branch June 30, 2025 22:30
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.

1 participant