chore(strip): drop this repository's process machinery in every strip variant - #258
Merged
Merged
Conversation
koniz-dev
force-pushed
the
chore/strip-process-artifacts
branch
from
September 26, 2026 15:28
42679ec to
1b09b3c
Compare
… variant The template shipped 31 MB of docs/verification against 1 MB of lib/, plus guards that enforce this repository's own issue workflow. A fork inherits neither epics nor these issue numbers, so issue-refs.yml would reject every pull request the adopter opens and check_epic_coverage would fail their first flutter test. Strip now drops those in all three variants, marks process-only sections in place rather than in a second list that can rot, and strip-smoke.yml asserts the removals happened - analyze and test stay green if the strip silently stops. CLAUDE.md gains the scope test that produced this change. Nothing is deleted from this repository; only the stripper learns to drop it. Refs #259
koniz-dev
force-pushed
the
chore/strip-process-artifacts
branch
from
September 26, 2026 15:33
1b09b3c to
7ac259e
Compare
koniz-dev
added a commit
that referenced
this pull request
Sep 26, 2026
…ss-only artifacts (#260) Verification phase for 8b80f2d (PR #258), which was merged without evidence, so gate G4 did not hold and the issue stayed open. All eight criteria were driven here; nothing routed to needs-uat. The three strip variants were run against pristine exports of 8b80f2d and each stripped tree passes pub get, analyze, test and check_docs with zero broken links. Two counterfactuals back the claims that an assertion alone cannot: strip-smoke.yml's assertion step, extracted verbatim, exits 1 on an unstripped tree while audit_template.sh exits 0 on that same tree; and an unbalanced process-only marker aborts the strip with exit 2 before any deletion, with before/after sentinels showing all 1003 docs/verification files intact. Refs #259 Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
A measurement of what this template actually ships:
lib/is 1.0 MB,docs/verification/alone is 31 MB. 97% of the documentation an adopter inherits is evidence about issues closed in this repository. Two of the guards they inherit do not merely take up space, they fail outright in a fork.Two changes, both requested directly by the repository owner.
There is no issue for this work. The owner asked for it directly and asked that no issue be filed just to have something to reference, so the commit carries no
Refs koniz-dev/flutter-starter#N. Consequence, stated plainly: the Issue refs check on this PR is red for exactly that reason, and for no other —check_issue_refs.shhas no exemption path. There is no auto-closing keyword anywhere in the commit or this body; the failure is the missingRefsline only. See "Open question" at the bottom.Change 1 - the strip drops process-only artifacts
The test applied to every candidate is the owner's: would an adopter notice if this were missing?
Stripped, with reasoning per item
docs/verification/scripts/test/run_acceptance.shsurvives and recreates the directory for their issues. Strip.tool/check_epic_coverage.dart+test/tooling/epic_coverage_test.dartepic:*label. An aRefs #259