Outcome: done
Changed: src/typosaurus_sandbox/research/engine.py:108-111,156-163,173-177, src/typosaurus_sandbox/research/pipeline.py:20-23,187-234,296-312, tests/test_research_engine.py:186-290, tests/test_research_pipeline.py:292-354
Verified by: verify() -> "make verify" exit_code 0, "Ran 226 tests OK verification passed"; only pre-existing StarletteDeprecationWarning from fastapi/testclient.py import in tests/test_api.py, none introduced
Findings: Retry: pipeline.py:192-211 retries RsearchError status>=500 with backoff min(0.5*2^(attempt-1),8.0) up to RETRY_MAX_ATTEMPTS=3 (pipeline.py:20-23); status<500 or None not retried. Unexpected exceptions recorded as failure WorkOutcome in consume (pipeline.py:296-312), appended to outcomes. Closure gated on requests_failed==0 (engine.py:159-163); report.closed=requests_failed==0 (engine.py:177); failed round prevents closure, later clean round keeps report.closed False (tests/test_research_engine.py:229-290). Deep/ai wired: every popped query yields WorkItem("web",query,deep=True,ai=True) (engine.py:111); pipeline passes deep/ai to client.search and cache probe (pipeline.py:157-158,170-176); test proves seed+subtopic web calls all carry deep=True and ai=True (tests/test_research_engine.py:203-222). Fix committed as 472a59e; this node's working-tree delta is only trailing-blank cleanup; no new dependencies, client.py/cache.py/config.py/frontier.py untouched.
Open: none
Confidence: high - all six acceptance criteria met with direct
Typosaurus-Run: 4e2afb673c7f4578a12276d9181b982d
Typosaurus-Node: e6071c7595ab487b84f99476243abe28
Typosaurus-Agent: @nadia
Refs: #31
Outcome: done
Changed: none
Verified by: PYTHONPATH=src python3 -m unittest discover -s tests -q -> "Ran 226 tests in 0.634s OK", EXIT_CODE=0
Findings: Criterion 1 PASS - grep for unittest.skip|skipIf|skipUnless|SkipTest|expectedFailure|pytest.mark.skip|pytest.skip|xfail|@skip|@disabled|pytestmark across tests/ returned 0 hits; case-insensitive skipif|skipunless|onlyif|not implemented also 0; runtime report shows no skipped/expected-failure suffix
Findings: Criterion 2 PASS - grep '^\s*(pass|\.\.\.)\s*$' returned 0 hits; AST scan of all 226 test_* functions found none with only-pass body and every one contains >=1 assertion (bare assert or self.assert*/fail* call)
Findings: Criterion 3 PASS - all 25 broad 'skip' grep hits individually inspected and are duplicates_skipped/cache counters or test names, not directives: tests/test_research_engine.py:132-136, tests/test_research_dedup.py:105-233, tests/test_research_scheduling.py:226-496, tests/test_research_client.py:426,434; bare 'return' at tests/test_research_scheduling.py:149,175,289 are worker loop-exit control flow (assertions at 157-162,183-186,298-300); tests/test_research_pipeline.py:333 tests exception handling with assertions at 344-351
Findings: Criterion 4 PASS - evidence recorded as file:line references above and stored in tree finding
Findings: Supplemental sweep for __test__|no cover|pragma|.skip(|mark. returned 0 hits; suite has grown to 226 tests (previous run 219) with no skipped/expected failures reported
Open
Typosaurus-Run: 4e2afb673c7f4578a12276d9181b982d
Typosaurus-Node: 46ed07b2395240b297e0fedbe3b672cd
Typosaurus-Agent: @tanya
Refs: #31
Outcome: done
Changed: tests/test_research_integration.py:1-70
Verified by: make verify -> exit_code 0, 199 tests OK (1 new integration test), "verification passed"; only pre-existing StarletteDeprecationWarning from fastapi/testclient.py, none introduced; standalone run of tests.test_research_integration -> 1 test OK in 0.386s
Findings:
- Live smoke before writing: GET https://rsearch.app.molodetz.nl/search?query=python+asyncio&count=2&content=true -> 200, success=true, 2 results, first https://docs.python.org/3/library/asyncio.html with 2153-char content, ~1.5s
- TestLiveResearchProbe invokes ResearchPipeline.run() on one bounded web WorkItem ("python asyncio") with cache=true, default_count=2, max_concurrency=2, request_timeout 30s, wrapped in asyncio.wait_for(60s) so a hung run fails rather than blocking the suite; no skip decorator
- First run asserts requests_succeeded>=1, urls_found>=1, contents_seen>=1, and zero client-cache hits; second run on the same RsearchClient asserts cache_hit=True (pipeline.py:151 _probe_cache -> client.py:60 search_cached) and exactly zero additional network requests, proving the cache=true path end-to-end
- Only-rsearch enforcement: config.base_url asserted == https://rsearch.app.molodetz.nl (config.py:12) and every urllib.request.urlopen full_url recorded by a wrapper asserted startswith that base, plus at least one /search contact
- Non-empty result derived from live responses asserted via frontier.snapshot() urls_seen>=1 and content_seen
Typosaurus-Run: 4e2afb673c7f4578a12276d9181b982d
Typosaurus-Node: 101f25665b934f2fb52b11cbe0a4c7e8
Typosaurus-Agent: @sveta
Refs: #31
Outcome: done
Changed:
src/typosaurus_sandbox/presentation/api/v1/calculator.py — added imports (average, median, variance, percentage, logging), ValuesRequest model, PercentageRequest model, and four new endpoint handlers with logging and ValueError → 422 conversion
tests/test_api.py — added 26 test cases across four new test classes
Verified by: `make verify` — 93 tests, 0 failures, 0 new warnings
Findings:
- `ValuesRequest` model accepts `values: list[float]` and serves /average, /median, /variance
- `PercentageRequest` model accepts `value: float, total: float` for /percentage
- All four endpoints log at DEBUG level on invocation
- Empty list (average, median, variance) and zero total (percentage) produce HTTP 422 with descriptive detail
Open: none
Confidence: high — 26 new tests pass, zero regressions, pattern matches existing endpoint conventions
Typosaurus-Run: 34b5946ec981488091bee588eb919ff2
Typosaurus-Node: 644d226aefca431c81cf8be8a46d2070
Typosaurus-Agent: @nadia
Refs: #28
Outcome: done
Changed: app/__init__.py:11-14 | src/calculator.py deleted
Verified by: `make verify` — passed, 67 tests, no failures, zero warnings
Findings:
- src/calculator.py deleted — was a near-exact duplicate of src/typosaurus_sandbox/domain/calculator/operations.py
- app/__init__.py imports now use typosaurus_sandbox.domain.calculator (canonical module) instead of src.calculator
- All 67 existing tests pass with no regressions
- Flask app continues to serve its HTML frontend via the same routes
Open: none
Confidence: high — syntactic correctness verified by `py_compile`, functional correctness verified by `make verify` passing all 67 tests
Typosaurus-Run: 34b5946ec981488091bee588eb919ff2
Typosaurus-Node: 467304eb286a457189bfd6050e235fcf
Typosaurus-Agent: @nadia
Refs: #28
Outcome: done
Changed: tests/test_api.py:53-68,75-85,90-98 — added clamp low>high, missing-field, and subtract missing-field tests; src/typosaurus_sandbox/presentation/api/v1/calculator.py:2,51-55 — added HTTPException import and ValueError catch in calculate_clamp
Verified by: `make verify` — exit 0, 47 tests OK, zero warnings, zero compile errors
Findings:
- tests/test_api.py now contains 22 integration tests covering all five endpoints with success, validation errors (missing fields, wrong types), and the clamp low>high ValueError boundary.
- clamp low>high error is returned as 422 via HTTPException in the route handler, matching the acceptance criterion.
- All 47 tests (25 unit + 22 integration) pass with zero warnings.
Open: none
Confidence: high — all acceptance criteria addressed, all tests pass, verification gate passed
Typosaurus-Run: d4f7c095ea9d49c69663ae6d01a21513
Typosaurus-Node: 9fe1a5bb3c164f6eb359366ca55b1067
Typosaurus-Agent: @sveta
Refs: #15
