Skip to content

Tier-3 batch: pagination helper, single scaffolder, AuditLink.for_model/independent_session, FastAPI bound, durable-transport docs (#513 #509 #523 #516 #502) - #567

Merged
antosubash merged 16 commits into
mainfrom
chore/tier3-batch-502-523
Oct 21, 2026
Merged

antosubash merged 16 commits into
mainfrom
chore/tier3-batch-502-523

Conversation

@antosubash

Copy link
Copy Markdown
Owner

Summary

Closes #513
Closes #509
Closes #523
Closes #516
Closes #502

Verification

  • Local CI, after merging main:
    • make lint passes.
    • make test-py: 5112 passed, 22 skipped.
    • make test-js: 672 passed.
    • make ci-doctor and make ci-css-build pass.
  • /ship (2 rounds):
    • Optimize made one cleanup commit.
    • Round 1 review was clean after 2 passes, with 8 fixes in total; an independent Codex review contributed two of them, the count query's execution options and the tenants page size.
    • Browser QA found 3 bugs and fixed them all: the outbox view returned 422 on a bad page, the JSON APIs had no page cap, and the users admin API didn't validate at all.
    • Round 2 review was clean after 2 passes, and a targeted re-test of the round-2 changes was clean.
  • Reports:

Report links are Claude Artifacts and start private — the author can share them from claude.ai/code/artifacts if reviewers need access.

Known limitations (accepted)

  • Irregular plurals such as movies and houses are not singularised correctly when scaffolding.
  • Four pagers keep their own UI rather than PaginationBar: users, settings, the file storage selection footer, and tenants (which only knows whether there is a next page).
  • paginate does not support joinedload on collections.

Test plan

  • CI is green
  • Audit log and background task tables page correctly with PaginationBar

https://claude.ai/code/session_01M9neheZZEe3sVpDi2S3zT4

…st (#516); file_storage tasks extra and transport-neutral durable docs (#502)

Claude-Session: https://claude.ai/code/session_01M9neheZZEe3sVpDi2S3zT4
…LI templates (#509)

make new-module now calls create_module(framework_repo=True) from the CLI
instead of ~980 lines of f-string templates. Both scaffolders share
templates/module/ and emit constants.py, models, contracts, service, deps,
guarded endpoints, Browse/Create/Edit pages, menu item with label_key,
permissions and locales, so a fresh module passes make lint. The in-repo
overlay adds workspace-sourced pyproject metadata, LICENSE, README and
i18n-keyed pages; framework_repo.wire_module wires host/root pyproject and the
publish-pypi matrix. Scaffold tests consolidated in framework/cli/tests.

Claude-Session: https://claude.ai/code/session_01M9neheZZEe3sVpDi2S3zT4
independent_session rolls back explicitly only on Exception; on cancellation
closing the session discards the transaction without an await that could
mask the original error.

Claude-Session: https://claude.ai/code/session_01M9neheZZEe3sVpDi2S3zT4
…-523

# Conflicts:
#	CHANGELOG.md
#	modules/api_tokens/api_tokens/resolver.py
#	modules/file_storage/file_storage/endpoints/views.py
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Deploying simple-module-python with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0f93657
Status: ✅  Deploy successful!
Preview URL: https://4d6359b8.simple-module-python.pages.dev
Branch Preview URL: https://chore-tier3-batch-502-523.simple-module-python.pages.dev

View logs

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
🔒 Security Review ✅ Completed 2026-10-21T14:10:06.780902Z c735eb3 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@antosubash
antosubash merged commit 5407c4f into main Oct 21, 2026
16 checks passed
@antosubash
antosubash deleted the chore/tier3-batch-502-523 branch October 21, 2026 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment