Skip to content

5 feature bootstrap without framework#10

Merged
lesnik512 merged 2 commits into
mainfrom
5-feature-bootstrap-without-framework
Mar 21, 2025
Merged

5 feature bootstrap without framework#10
lesnik512 merged 2 commits into
mainfrom
5-feature-bootstrap-without-framework

Conversation

@lesnik512
Copy link
Copy Markdown
Member

No description provided.

@lesnik512 lesnik512 self-assigned this Mar 21, 2025
@lesnik512 lesnik512 linked an issue Mar 21, 2025 that may be closed by this pull request
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Flag Coverage Δ
unittests 100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
lite_bootstrap/bootstraps/fastapi_bootstrap.py 100.00% <100.00%> (ø)
lite_bootstrap/bootstraps/free_bootstrap.py 100.00% <100.00%> (ø)
tests/test_free_bootstrap.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lesnik512 lesnik512 merged commit a8f7ee6 into main Mar 21, 2025
@lesnik512 lesnik512 deleted the 5-feature-bootstrap-without-framework branch March 21, 2025 10:35
lesnik512 added a commit that referenced this pull request Jun 2, 2026
PR #107 (instrument skip rework) shipped a few days after the original
arc closed. Captured three new datapoints worth recording:

- Mid-design pivot worked: user pushback on the is_configured arg
  forced the design through pre-#88 history and confirmed the
  classmethod-with-arg pattern was correct.
- Defensive workaround was wrong design: my _get_logger() fresh-per-
  call structlog proxy made tests pass but the user's pivot to stdlib
  logging + public build_summary() was the actual right answer. caplog
  (which the original plan flagged but the subagent ignored) was the
  correct test mechanism.
- LSP violations on classmethod parameter overrides are an emergent
  pattern: 3 framework instrument overrides needed
  # ty: ignore[invalid-method-override] because they narrow
  bootstrap_config covariantly. Existing field-narrowing pattern is
  accepted; method-param narrowing now needs the explicit ignore.

Added action items #8 (workaround-vs-framework-choice heuristic), #9
(cap subagent dispatch scope; the ~60-minute connection drop orphaned
work), and #10 (document the LSP override pattern in CLAUDE.md).

Reaffirms the original retro's closing observation: subagent loop
reliably produces a green-tests implementation of the spec, but the
spec is rarely the right design. Design emerges during review
iteration. PR #107 needed 5 user follow-up commits after the
mechanical migration landed.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feature] Bootstrap without framework

1 participant