Commit Graph
7 Commits
Author SHA1 Message Date
Parth 9faf788bd5 Merge pull request #16 from parththakkar106/refactor-17
Phase 17: split the four largest files, remove duplication, and drop the legacy columns
2026-08-30 19:01:40 +05:30
Parth e7d75c3b05 Rewrite Python comments in Google developer documentation style (#12)
* Rewrite comments in Google developer documentation style

Rewrite the comments and docstrings across the backend core modules so they
read plainly. The previous prose was accurate but dense and figurative, which
made it slow to skim.

Applies the Google developer documentation style guide: short sentences, active
voice, present tense, American spelling, and no metaphors, idioms, or
rhetorical asides. Replaces em-dash chains with separate sentences.
2026-08-26 15:37:25 +05:30
ParthandClaude Opus 5 0f1e05c808 Keep the fixture, so there is something to scroll (#5)
The harness already built a production-shaped 600-action adventure and then
threw it away with the temp file. The one open gap in plan/13 is that nothing
has ever driven the scroll in a browser, and part of why is that there was
never a long adventure to drive it with.

--keep PATH writes the fixture somewhere durable and makes the app able to
serve it. Two edits are needed for that, both of which cost an hour to
rediscover:

  - create_all() builds the current schema but leaves the version stamp at its
    default, and bootstrap() reads a populated-but-unstamped database as
    ancient — it replays every migration against a schema that already has the
    columns, and fails on the first.
  - the fixture's user is a registered one, but local mode looks for the row
    with email IS NULL and is_guest false, so without clearing the email the
    app opens on an empty library.

--keep is read before argparse exists, because where the database lives has to
be settled before app.database is imported. That is the same constraint the
AIDND_STRESS_DATABASE_URL block already lives under. SQLite only; combining it
with a Postgres target is rejected rather than half-honoured.

Verified end to end: the fixture boots with no manual step, action_count 600,
a 60-action first payload, and before_id walks back nine more pages to the
start. Nothing about the default path changed; 259 tests pass.


Claude-Session: https://claude.ai/code/session_017Dvvqn9ZDR4ixeFPHNbww7

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
2026-08-17 16:50:41 +05:30
Parth 78d73b94ea Merge pull request #4 from parththakkar106/measure-post-vacuum-sizes
Measure what the vacuum actually left behind
2026-08-17 16:19:41 +05:30
Parth 70ce936e10 Merge pull request #3 from parththakkar106/handover-2026-08-17
Update STATUS after the egress work shipped
2026-08-17 15:42:18 +05:30
Parth 13a5dd3f9e Merge pull request #2 from parththakkar106/fix-prepend-autoscroll
Fix: loading earlier turns jumped to the end of the story
2026-08-17 15:23:40 +05:30
Parth b4af5a81f9 Cut database egress and storage
Cut database egress and storage: window the story, compress the prompts
2026-08-17 15:10:08 +05:30