fix(datastore): serialize native bucket datetimes

[TimeToBuildBob]

Summary

• accept Peewee's native datetime value when serializing bucket metadata
• keep parsing string values for compatibility with older/storage-dependent rows
• normalize the result to UTC ISO 8601 as before

Verification

• reproduced BucketModel.json() crashes with iso8601.ParseError when 'created' is a datetime object #157 on current master: test_create_bucket[datastore0] failed with iso8601.ParseError
• 232 passed, 2 skipped
• ruff check .
• ruff format --check aw_datastore/storages/peewee.py

The existing test_create_bucket already covers this exact path and failed before the fix, so this PR does not add duplicate coverage.

Fixes #157

[greptile-apps]

Confidence Score: 5/5

The PR appears safe to merge, with no actionable correctness, security, or repository-rule issues identified.

Summary

This PR updates Peewee bucket metadata serialization to accept both native datetime values and persisted string values while retaining UTC ISO 8601 output.

• Parses legacy or storage-dependent string values as before.
• Serializes native Peewee datetime values without passing them to the string parser.
• Preserves the shared bucket metadata shape and UTC normalization.

_{Reviews (1) · Last reviewed commit: "fix(datastore): serialize native bucket ..."}

[TimeToBuildBob]

CI-green and mergeable (Greptile 5/5) — waiting only on a maintainer click.

This PR is ready to merge, but the bot has pull-only access to this repo and can't self-merge — surfacing it here so it isn't lost. The monitoring loop will stop re-flagging it now that this note is posted.

[TimeToBuildBob]

🤖 AI code review

The PR modifies BucketModel.json() in aw_datastore/storages/peewee.py to accept Peewee's native datetime value for the 'created' field, while still parsing string values for compatibility. It assigns self.created to a local variable, converts it to a datetime if it is a string, and then normalizes to UTC ISO 8601 as before.

Safe to merge — no P0/P1 findings

Confidence 5/5

✅ No thread-worthy findings. Advisory notes follow; they are retained without opening review threads.

1 advisory finding (summary-only, not scored)

These P2 guard, heuristic, trade-off, or documentation claims are retained for judgment without opening review threads.

⚠️ P2 medium — aw_datastore/storages/peewee.py

This is a fix(...) PR but no test files are included in the diff. Erik's feedback: 'where is the repro & fixes they are supposed to catch' (gptme#3441), 'that measurement should come with a regression test' (gptme#3446). Add a test that would have caught this bug. (Advisory: Erik merged all such PRs but consistently requested tests.)

Add a test file that reproduces the bug before the fix and passes after it.

_{How this was verified: static preflight: fix-commit + touched-files scan (rule 7)}

Files changed (1) — the diff as I read it

• aw_datastore/storages/peewee.py — Changes BucketModel.json() to handle both datetime and string 'created' values by converting strings via iso8601.parse_date before UTC normalization.

_{Reviewed 6d5aae47896c · openrouter/deepseek/deepseek-v4-flash-0731 · llm engine · 6s · about this reviewer}

_{Maintainer commands}

_{@TimeToBuildBob review (own line) — fresh review · @TimeToBuildBob fix — a worker acts on the findings. Once per comment; 👀 = received.}