nohype/main-site
5
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Implement implementation.md end-to-end (containerized Django/Wagtail) #1

Closed
codex_b wants to merge 0 commits from codex_b/implementation-e2e into main
pull from: codex_b/implementation-e2e
merge into: nohype:main
Conversation 0 Commits - Files Changed - +2323
codex_b commented 2026-02-28 12:23:42 +00:00
Owner
Copy Link

Summary\n- implemented the full Django/Wagtail project scaffold and app architecture from \n- added containerized development/test setup (, ) with split settings for development/production\n- implemented core domains: blog pages + blocks + feeds, authors snippet, legal pages, consent system, comments flow, newsletter subscribe/confirm\n- added templates/static baseline (home, index, article, about, legal, shared components)\n- added CI workflow and project docs/changelog\n\n## Key Engineering Details\n- all verification is Docker-based (no bare-metal test execution)\n- read-time calculation implemented on article save and excludes code blocks\n- tag metadata styling support implemented via template filter and snippet model\n- consent modeled as URL-encoded cookie state with middleware injection\n- comments include moderation-ready model + rate limiting + server-side enforcement\n- newsletter confirmation links now enforce token max-age\n\n## Validation\n- ....................................... [100%]

=============================== warnings summary ===============================
apps/blog/tests/test_feeds.py: 1 warning
apps/blog/tests/test_feeds_more.py: 1 warning
apps/blog/tests/test_views.py: 3 warnings
apps/comments/tests/test_more.py: 1 warning
apps/comments/tests/test_views.py: 2 warnings
apps/core/tests/test_consent.py: 1 warning
apps/core/tests/test_tags.py: 1 warning
apps/legal/tests/test_models.py: 2 warnings
apps/newsletter/tests/test_views.py: 4 warnings
/usr/local/lib/python3.12/site-packages/whitenoise/base.py:115: UserWarning: No directory at: /app/staticfiles/
warnings.warn(f"No directory at: {root}")

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html

---------- coverage: platform linux, python 3.12.12-final-0 ----------
Name Stmts Miss Cover Missing

apps/init.py 0 0 100%
apps/authors/init.py 0 0 100%
apps/authors/apps.py 4 0 100%
apps/authors/models.py 26 1 96% 43
apps/authors/wagtail_hooks.py 10 0 100%
apps/blog/init.py 0 0 100%
apps/blog/apps.py 4 0 100%
apps/blog/blocks.py 37 3 92% 28-30
apps/blog/feeds.py 28 3 89% 24, 38, 41
apps/blog/models.py 110 6 95% 66, 73-74, 146-147, 166
apps/blog/wagtail_hooks.py 8 0 100%
apps/comments/init.py 0 0 100%
apps/comments/apps.py 4 0 100%
apps/comments/forms.py 14 1 93% 18
apps/comments/models.py 19 1 95% 25
apps/comments/urls.py 3 0 100%
apps/comments/views.py 35 4 89% 30, 35, 41-42
apps/comments/wagtail_hooks.py 14 1 93% 17
apps/core/init.py 0 0 100%
apps/core/apps.py 4 0 100%
apps/core/consent.py 30 2 93% 39-40
apps/core/context_processors.py 6 0 100%
apps/core/middleware.py 8 0 100%
apps/core/models.py 7 0 100%
apps/core/templatetags/init.py 0 0 100%
apps/core/templatetags/core_tags.py 23 0 100%
apps/core/templatetags/seo_tags.py 18 2 89% 21-22
apps/core/views.py 23 7 70% 11, 19-24, 33
apps/legal/init.py 0 0 100%
apps/legal/apps.py 4 0 100%
apps/legal/models.py 17 0 100%
apps/newsletter/init.py 0 0 100%
apps/newsletter/apps.py 4 0 100%
apps/newsletter/forms.py 5 0 100%
apps/newsletter/models.py 8 1 88% 11
apps/newsletter/services.py 13 0 100%
apps/newsletter/urls.py 3 0 100%
apps/newsletter/views.py 37 3 92% 21, 45-46

TOTAL 526 35 93%

Required test coverage of 90% reached. Total coverage: 93.35%
39 passed, 16 warnings in 21.27s\n - result: 39 passed\n - coverage: 93.35% (threshold 90%)\n- All checks passed!\n - result: passed\n- Success: no issues found in 70 source files\n - result: passed\n\n## Commits\n- b5f0f40 Scaffold containerized Django/Wagtail app with core features\n- 8970f4d Add Docker-executed pytest suite with >90% coverage\n- 938ff5b Add CI workflow and project runbook documentation\n\n## Notes\n- dependency compatibility was validated and adjusted for Wagtail 7 by pinning \n- performed an additional focused review pass on consent/comments/newsletter critical paths and addressed findings

## Summary\n- implemented the full Django/Wagtail project scaffold and app architecture from \n- added containerized development/test setup (, ) with split settings for development/production\n- implemented core domains: blog pages + blocks + feeds, authors snippet, legal pages, consent system, comments flow, newsletter subscribe/confirm\n- added templates/static baseline (home, index, article, about, legal, shared components)\n- added CI workflow and project docs/changelog\n\n## Key Engineering Details\n- all verification is Docker-based (no bare-metal test execution)\n- read-time calculation implemented on article save and excludes code blocks\n- tag metadata styling support implemented via template filter and snippet model\n- consent modeled as URL-encoded cookie state with middleware injection\n- comments include moderation-ready model + rate limiting + server-side enforcement\n- newsletter confirmation links now enforce token max-age\n\n## Validation\n- ....................................... [100%] =============================== warnings summary =============================== apps/blog/tests/test_feeds.py: 1 warning apps/blog/tests/test_feeds_more.py: 1 warning apps/blog/tests/test_views.py: 3 warnings apps/comments/tests/test_more.py: 1 warning apps/comments/tests/test_views.py: 2 warnings apps/core/tests/test_consent.py: 1 warning apps/core/tests/test_tags.py: 1 warning apps/legal/tests/test_models.py: 2 warnings apps/newsletter/tests/test_views.py: 4 warnings /usr/local/lib/python3.12/site-packages/whitenoise/base.py:115: UserWarning: No directory at: /app/staticfiles/ warnings.warn(f"No directory at: {root}") -- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html ---------- coverage: platform linux, python 3.12.12-final-0 ---------- Name Stmts Miss Cover Missing ------------------------------------------------------------------- apps/__init__.py 0 0 100% apps/authors/__init__.py 0 0 100% apps/authors/apps.py 4 0 100% apps/authors/models.py 26 1 96% 43 apps/authors/wagtail_hooks.py 10 0 100% apps/blog/__init__.py 0 0 100% apps/blog/apps.py 4 0 100% apps/blog/blocks.py 37 3 92% 28-30 apps/blog/feeds.py 28 3 89% 24, 38, 41 apps/blog/models.py 110 6 95% 66, 73-74, 146-147, 166 apps/blog/wagtail_hooks.py 8 0 100% apps/comments/__init__.py 0 0 100% apps/comments/apps.py 4 0 100% apps/comments/forms.py 14 1 93% 18 apps/comments/models.py 19 1 95% 25 apps/comments/urls.py 3 0 100% apps/comments/views.py 35 4 89% 30, 35, 41-42 apps/comments/wagtail_hooks.py 14 1 93% 17 apps/core/__init__.py 0 0 100% apps/core/apps.py 4 0 100% apps/core/consent.py 30 2 93% 39-40 apps/core/context_processors.py 6 0 100% apps/core/middleware.py 8 0 100% apps/core/models.py 7 0 100% apps/core/templatetags/__init__.py 0 0 100% apps/core/templatetags/core_tags.py 23 0 100% apps/core/templatetags/seo_tags.py 18 2 89% 21-22 apps/core/views.py 23 7 70% 11, 19-24, 33 apps/legal/__init__.py 0 0 100% apps/legal/apps.py 4 0 100% apps/legal/models.py 17 0 100% apps/newsletter/__init__.py 0 0 100% apps/newsletter/apps.py 4 0 100% apps/newsletter/forms.py 5 0 100% apps/newsletter/models.py 8 1 88% 11 apps/newsletter/services.py 13 0 100% apps/newsletter/urls.py 3 0 100% apps/newsletter/views.py 37 3 92% 21, 45-46 ------------------------------------------------------------------- TOTAL 526 35 93% Required test coverage of 90% reached. Total coverage: 93.35% 39 passed, 16 warnings in 21.27s\n - result: 39 passed\n - coverage: 93.35% (threshold 90%)\n- All checks passed!\n - result: passed\n- Success: no issues found in 70 source files\n - result: passed\n\n## Commits\n- b5f0f40 Scaffold containerized Django/Wagtail app with core features\n- 8970f4d Add Docker-executed pytest suite with >90% coverage\n- 938ff5b Add CI workflow and project runbook documentation\n\n## Notes\n- dependency compatibility was validated and adjusted for Wagtail 7 by pinning \n- performed an additional focused review pass on consent/comments/newsletter critical paths and addressed findings
codex_b added 3 commits 2026-02-28 12:23:42 +00:00
Scaffold containerized Django/Wagtail app with core features b5f0f40c4c
Add Docker-executed pytest suite with >90% coverage 8970f4d8de
Add CI workflow and project runbook documentation
Some checks failed
CI / test (push) Failing after 2m52s
Details
CI / test (pull_request) Failing after 2m32s
Details
938ff5b0d2
codex_b closed this pull request 2026-02-28 12:23:56 +00:00
Some checks are pending
CI / test (push) Failing after 2m52s
Details
CI / test (pull_request) Failing after 2m32s
Details
CI / pr-e2e (pull_request)
Required
CI / ci (pull_request)
Required

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claude codex_a codex_b codex_c mark
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: nohype/main-site#1
Delete Branch "codex_b/implementation-e2e"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?