refactor(management): clean up structural indirection across lib #110

New issue

Closed

opened 2026-09-28 19:18:05 +00:00 by yuris · 0 comments

yuris commented

2026-09-28 19:18:05 +00:00

Owner

Problem

apps/management contains structural indirection across types and messaging modules that increases reader load:

apps/management/lib/types/index.ts is an artificial barrel re-exporting domain types, causing callers to bypass direct domain slice paths.
apps/management/lib/connectors/whatsapp-wwebjs/ and apps/management/lib/messaging/ fragment single-service WhatsApp handling across redundant directories and a one-caller adapter layer.
Narrative comments, obsolete divider banners, and redundant method signatures in apps/management/lib/ create clutter.

Acceptance Criteria

Delete apps/management/lib/types/index.ts and migrate every caller to direct domain slice imports (@/lib/orders/types, @/lib/customers/types, @/lib/inventory/types, @/lib/purchases/types).
Consolidate apps/management/lib/connectors/whatsapp-wwebjs/ and apps/management/lib/messaging/ into a single cohesive apps/management/lib/whatsapp/ directory. Collapse one-caller adapter layers and remove obsolete directories.
Run a no-comments pass over apps/management/lib/ to delete narrative narration, obsolete headers, and commented-out corpses.
Verify with pnpm verify:management.

## Problem `apps/management` contains structural indirection across types and messaging modules that increases reader load: 1. `apps/management/lib/types/index.ts` is an artificial barrel re-exporting domain types, causing callers to bypass direct domain slice paths. 2. `apps/management/lib/connectors/whatsapp-wwebjs/` and `apps/management/lib/messaging/` fragment single-service WhatsApp handling across redundant directories and a one-caller adapter layer. 3. Narrative comments, obsolete divider banners, and redundant method signatures in `apps/management/lib/` create clutter. ## Acceptance Criteria 1. Delete `apps/management/lib/types/index.ts` and migrate every caller to direct domain slice imports (`@/lib/orders/types`, `@/lib/customers/types`, `@/lib/inventory/types`, `@/lib/purchases/types`). 2. Consolidate `apps/management/lib/connectors/whatsapp-wwebjs/` and `apps/management/lib/messaging/` into a single cohesive `apps/management/lib/whatsapp/` directory. Collapse one-caller adapter layers and remove obsolete directories. 3. Run a no-comments pass over `apps/management/lib/` to delete narrative narration, obsolete headers, and commented-out corpses. 4. Verify with `pnpm verify:management`.

yuris referenced this issue from a pull request that will close it,

2026-09-28 19:20:51 +00:00

refactor(management): clean up structural indirection across lib #111

yuris closed this issue

2026-09-28 19:27:23 +00:00

yuris referenced this issue

2026-09-28 20:03:54 +00:00

refactor(management): replace LangGraph with native PostgreSQL state runner #113