knopersikcuo/Planner
Watch
1
0
Fork
You've already forked Planner
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Clean up dead and duplicated repository code #108

Merged
kleb merged 1 commit from chore/repository-cleanup-2026-08-23 into main 2026-08-23 16:56:50 +02:00
Conversation 1 Commits 1 Files changed 20 +201 -279
kleb commented 2026-08-23 16:55:37 +02:00
Collaborator
Copy link

Summary

  • remove a dead memory-search facade and register the dynamically loaded API test config with Knip
  • break PostgreSQL operator CLI import cycles and consolidate behavior-neutral server helpers
  • share repeated task and collaboration UI plumbing while retaining persistence, recovery, credential, and domain boundaries

Verification

  • bun run typecheck
  • bun run lint
  • bun run lint:react
  • bun run format:check
  • bun run knip
  • bun test (909/909)
  • node ./node_modules/vitest/vitest.mjs run --maxWorkers=4 (216/216)
## Summary - remove a dead memory-search facade and register the dynamically loaded API test config with Knip - break PostgreSQL operator CLI import cycles and consolidate behavior-neutral server helpers - share repeated task and collaboration UI plumbing while retaining persistence, recovery, credential, and domain boundaries ## Verification - `bun run typecheck` - `bun run lint` - `bun run lint:react` - `bun run format:check` - `bun run knip` - `bun test` (909/909) - `node ./node_modules/vitest/vitest.mjs run --maxWorkers=4` (216/216)
kAgent commented 2026-08-23 16:55:37 +02:00
Collaborator
Copy link

kReview review

Verdict: no findings

No findings to address in the reviewed diff.

No concrete correctness, security, data-integrity, concurrency, or performance regressions are established by the visible changes and repository context.

Reviewed by kReview at 17df75c1cf. This comment is conservative and based only on the PR diff, metadata, and supplied repository context.

Est. cost ~$0.21 (43.9k in / 357 out / gpt-5.6-sol).

<!-- codex-forgejo-review --> <!-- codex-forgejo-review-head:17df75c1cf260241faa807f5b3d339a5a3a5e07f --> ## kReview review **Verdict:** no findings No findings to address in the reviewed diff. No concrete correctness, security, data-integrity, concurrency, or performance regressions are established by the visible changes and repository context. _Reviewed by kReview at `17df75c1cf`. This comment is conservative and based only on the PR diff, metadata, and supplied repository context._ _Est. cost ~$0.21 (43.9k in / 357 out / gpt-5.6-sol)._ <!-- codex-forgejo-review-state:eyJoZWFkU2hhIjoiMTdkZjc1YzFjZjI2MDI0MWZhYTgwN2Y1YjNkMzM5YTVhM2E1ZTA3ZiIsInN1bW1hcnkiOiJObyBjb25jcmV0ZSBjb3JyZWN0bmVzcywgc2VjdXJpdHksIGRhdGEtaW50ZWdyaXR5LCBjb25jdXJyZW5jeSwgb3IgcGVyZm9ybWFuY2UgcmVncmVzc2lvbnMgYXJlIGVzdGFibGlzaGVkIGJ5IHRoZSB2aXNpYmxlIGNoYW5nZXMgYW5kIHJlcG9zaXRvcnkgY29udGV4dC4iLCJmaW5kaW5ncyI6W10sImN1bXVsYXRpdmVDb3N0Ijp7InVzZCI6MC4yMTM2NjEsImlucHV0VG9rZW5zIjo0MzkzMSwib3V0cHV0VG9rZW5zIjozNTcsIm1vZGVsIjoiZ3B0LTUuNi1zb2wifX0= -->
kleb merged commit b694a3b56a into main 2026-08-23 16:56:50 +02:00
kleb deleted branch chore/repository-cleanup-2026-08-23 2026-08-23 16:56:50 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
knopersikcuo/Planner!108
No description provided.