Skip to content

Refactor API boundaries and desloppify cleanup across backend and dashboard flows#49

Merged
Feikovandijk merged 12 commits intomainfrom
fix/desloppify-progress-20260310
Mar 31, 2026
Merged

Refactor API boundaries and desloppify cleanup across backend and dashboard flows#49
Feikovandijk merged 12 commits intomainfrom
fix/desloppify-progress-20260310

Conversation

@Feikovandijk
Copy link
Copy Markdown
Owner

Summary

  • align backend and frontend API surfaces to reduce duplicated clients, payload shapes, and boundary inconsistencies
  • clean up auth and middleware noise by removing debug-heavy logging and trimming repetitive infrastructure docblocks
  • update dashboard and trends components to use shared types and valid chart configuration so the frontend builds cleanly
  • add project hygiene updates for desloppify workflow artifacts and document progress/output files
  • include follow-up backend script, controller, utility, and test updates tied to the code-health pass

Testing

  • cd api && npm test
  • cd api && npm run type-check
  • cd devdashboard && npm test
  • cd devdashboard && npm run type-check
  • cd devdashboard && npm run build
  • cd user-dashboard && npm test
  • cd user-dashboard && npm run type-check
  • cd user-dashboard && npm run build

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces the 'desloppify' tool documentation and progress tracking while implementing extensive code cleanup and type safety enhancements. Key improvements include replacing 'any' types with specific interfaces, removing verbose debug logs, and adding unit tests for configuration, middleware, and utility functions. Review feedback focuses on maintaining appropriate logging levels for critical errors, removing redundant empty middleware in the authentication flow, and refining type guards to ensure proper object narrowing in the dashboard statistics logic.

Feikovandijk pushed a commit that referenced this pull request Mar 31, 2026
@Feikovandijk Feikovandijk force-pushed the fix/desloppify-progress-20260310 branch from bd6e13a to 638f0f5 Compare March 31, 2026 07:55
@Feikovandijk Feikovandijk merged commit 16ed72b into main Mar 31, 2026
5 checks passed
@Feikovandijk Feikovandijk deleted the fix/desloppify-progress-20260310 branch March 31, 2026 08:32
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.

3 participants