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

Remove obsolete memory eval tooling and stabilize E2E tests #120

Merged
kleb merged 1 commit from chore/cleanup-scripts-tests into main 2026-08-26 12:40:18 +02:00
Conversation 1 Commits 1 Files changed 12 +19 -1814
kleb commented 2026-08-26 12:38:49 +02:00
Collaborator
Copy link

Summary

  • remove retired memory evaluation scripts, commands, and tests
  • remove the invalid in-memory server restart check and stabilize authenticated Playwright setup
  • fix stale E2E locators and accessibility naming, then refresh the current shell snapshots

Verification

  • bun run check:quality
  • bun run test:e2e
  • bun run e2e:shots

This covers 942 Bun tests, 224 Vitest tests, and 22 Playwright tests.

## Summary - remove retired memory evaluation scripts, commands, and tests - remove the invalid in-memory server restart check and stabilize authenticated Playwright setup - fix stale E2E locators and accessibility naming, then refresh the current shell snapshots ## Verification - `bun run check:quality` - `bun run test:e2e` - `bun run e2e:shots` This covers 942 Bun tests, 224 Vitest tests, and 22 Playwright tests.
kAgent commented 2026-08-26 12:38:51 +02:00
Collaborator
Copy link

kReview review

Verdict: no findings

No findings to address in the reviewed diff.

The visible changes present low review risk, with no concrete correctness, security, or test-stability defect established by the diff and available repository evidence.

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

Est. cost ~$1.12 (615.0k in / 1.4k out / gpt-5.6-sol).

<!-- codex-forgejo-review --> <!-- codex-forgejo-review-head:216efeb94e7a94e011f952e621754f40407e295e --> ## kReview review **Verdict:** no findings No findings to address in the reviewed diff. The visible changes present low review risk, with no concrete correctness, security, or test-stability defect established by the diff and available repository evidence. _Reviewed by kReview at `216efeb94e`. This comment is conservative and based only on the PR diff, metadata, and supplied repository context._ _Est. cost ~$1.12 (615.0k in / 1.4k out / gpt-5.6-sol)._ <!-- codex-forgejo-review-state:eyJoZWFkU2hhIjoiMjE2ZWZlYjk0ZTdhOTRlMDExZjk1MmU2MjE3NTRmNDA0MDdlMjk1ZSIsInN1bW1hcnkiOiJUaGUgdmlzaWJsZSBjaGFuZ2VzIHByZXNlbnQgbG93IHJldmlldyByaXNrLCB3aXRoIG5vIGNvbmNyZXRlIGNvcnJlY3RuZXNzLCBzZWN1cml0eSwgb3IgdGVzdC1zdGFiaWxpdHkgZGVmZWN0IGVzdGFibGlzaGVkIGJ5IHRoZSBkaWZmIGFuZCBhdmFpbGFibGUgcmVwb3NpdG9yeSBldmlkZW5jZS4iLCJmaW5kaW5ncyI6W10sImN1bXVsYXRpdmVDb3N0Ijp7InVzZCI6MS4xMTcwMDYwMDAwMDAwMDAyLCJpbnB1dFRva2VucyI6NjE0OTcyLCJvdXRwdXRUb2tlbnMiOjEzNTgsIm1vZGVsIjoiZ3B0LTUuNi1zb2wifX0= -->
kleb merged commit 51f370f929 into main 2026-08-26 12:40:18 +02:00
kleb deleted branch chore/cleanup-scripts-tests 2026-08-26 12:42:18 +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!120
No description provided.