Skip to content

feat(extraction): flip TRAWL_RS_TRAF default on#84

Merged
dongwhee merged 1 commit into
developfrom
feat/rs-traf-default-on
Jul 5, 2026
Merged

feat(extraction): flip TRAWL_RS_TRAF default on#84
dongwhee merged 1 commit into
developfrom
feat/rs-traf-default-on

Conversation

@dongwhee

@dongwhee dongwhee commented Jul 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

Follow-up to #83: the rs-trafilatura extraction candidate is now default ON (TRAWL_RS_TRAF=0 to opt out), matching the _RECORDS_ENABLED default-on convention. Still an optional dependency — a missing package logs one debug line and the candidate is skipped, so installs without rs-trafilatura see no behavior change from their perspective other than the flag semantics.

Gates (2026-07-04)

Gate Result
Parity, default env (fetch cache disabled to exercise the candidate) PASS 15/15
Parity, opt-out (TRAWL_RS_TRAF=0) PASS 15/15
agent_patterns coding shard, default env PASS 24/24
Offline import semantics (unset imports, =0 does not) + tests/test_extraction.py PASS

Changes

  • src/trawl/extraction.py: one-line default flip in _rs_traf_enabled().
  • .env.example / CHANGELOG.md: opt-out semantics.
  • CLAUDE.md: default-on feature list entry, including the measurement caveat that the C8 fetch cache stores post-extraction results (extraction A/Bs must set TRAWL_FETCH_CACHE_TTL=0).

🤖 Generated with Claude Code

rs-trafilatura extraction candidate is now default on (opt out via
TRAWL_RS_TRAF=0), matching the _RECORDS_ENABLED default-on convention.
Still an optional dependency: missing package logs one debug line and
the candidate is skipped.

Gates (2026-07-04):
- Parity 15/15 default env (fetch cache disabled to exercise the
  candidate) and 15/15 with TRAWL_RS_TRAF=0.
- agent_patterns coding shard 24/24 default env.
- tests/test_extraction.py pass; unset-env import / =0 no-import
  verified offline.

Docs: .env.example opt-out semantics, CHANGELOG entry amended,
CLAUDE.md default-on feature list entry (includes the fetch-cache
A/B measurement caveat).
@dongwhee dongwhee merged commit d5a6047 into develop Jul 5, 2026
4 checks passed
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.

1 participant