Growth: promote 5 lessons from the fleet's local packs - #497
Merged
Conversation
Refs #99 From the 2026-07-27 promote window (9 members, 7 of them newly minted local packs): - engineering-practices: `pkill -f` self-matches the invoking shell (EdFringeNow). - git-github-advanced: `git checkout <ref> -- <paths>` destroys uncommitted edits (GoogleCalendarEventCreator); probing an egress-blocked host with a throwaway push-triggered workflow (EdFringeNow). - basics: the tell that the SessionStart hook did not fire (GoogleCalendarEventCreator). - generate-project-instructions: a pack homes how we work, never a statement of what the product does (TLDR). - research-project: an overlay never uses the signal's own colour, and shows where the method breaks (VascularColoring). - flutter: don't commit lockfile churn from a local SDK (HelloWorldFlutterApp). All seven landed as prose: each is in-flight judgment or knowledge with no repo-state signature a check could carry.
This was referenced Jul 27, 2026
Review on #497: - The SessionStart-hook-didn't-fire rule is self-defeating where it landed. Its premise is "no pack prose reached context"; it IS pack prose, so in the only situation it describes it is not in context to be read. Removed rather than reworded — detecting a hook that didn't fire belongs in code, not in the payload the hook delivers. - The pubspec.lock rule read as if the file shouldn't be tracked. An app commits its lockfile (libraries gitignore it); the churn is the skew between a local SDK's resolution and CI's. Restated around the actual signature — lock moved, yaml didn't — and given the concrete revert. Refs #99
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
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.
Refs #99. The
growth-promoterun for 2026-07-27. Nine members in the dispatch Context; seven of them had their local packs minted since the last run, so the candidate pool was unusually large. Not auto-merged — owner approval.Two of the run's original seven were dropped in owner review (see the log at the bottom), leaving five.
packs/basics/skills/engineering-practices/SKILL.mdpkill -f "<pattern>"matches the invoking shell's own command line, so the shell dies and everything chained after it is silently dropped. Never chain a-fpkill; bracket a character ("[h]ttp.server 8099") to break the self-match.git-github-advancedgit checkout <ref> -- <paths>writes<ref>'s committed tree over the working copy, destroying the uncommitted edits it was reached for — unrecoverably. Branch in place, or commit/stash first.generate-project-instructionspacks/research-project/RULES.md§1packs/flutter/RULES.mdpubspec.lockmoving withoutpubspec.yamlmoving is version skew, not a dependency change — an app commits its lockfile, so revert the churn a local SDK's differing resolution produced rather than gitignoring the file.All five landed as prose. Four are in-flight behaviour (what the agent typed, which tool it reached for) or judgment (pack-worthiness) with no repo-state signature a conformance check could carry; the overlay-colour rule's grounded case is checkable, but only against a project's own render script — it is a local check in VascularColoring already, and generalizing the palette to canon has no target to scan. No conformance-backlog issues opened — these are prose-shaped, not deferred check work.
Notable rejections — already carried by the corpus
origin/mainmakes work-scoped checks flag commits already on main →git-github-advanced, "A conformance finding on history you didn't write may be a stale diff base".research-project§2, verbatim ("Ground truth is annotated, never fabricated… see §9 on validation tiers").engineering-practices(GENERATED discipline) andresearch-project§2.engineering-practices(single-source-of-truth).get_check_runs, notget_status; read a non-PR run's job logs; reach GitHub via MCP notcurl→git-github-advanced, three existing sections.basics/scheduled-tasks.md(rejected in the 2026-07-26 run too).Deferred, not rejected (real and portable, but the routing needs a new pack, which is its own PR and didn't fit this run's bound):
firebase.executablePath, import the ESM entry (EdFringeNow, measured at ~220s of rediscovery over five sessions). No browser-automation pack exists.Dropped in owner review
packs/basics/RULES.md) — self-defeating where it landed: its premise is "no pack prose reached context", but it is pack prose, so in the only situation it describes it is not in context to be read. Detecting a hook that didn't fire belongs in code, not in the payload the hook delivers.git-github-advanced) — eliminated.Stayed local as product-specific: LaughCounter's on-device audio boundary, ShoutsAndWhispers' presence/TTL and cross-tier constant table, EdFringeAllocator's year-folder and credential rules, EdFringeData's wire format and
ticketStatussemantics, gRatio's g-ratio invariants, GCEC's extractor pipeline and architecture rules, TLDR's three version records.Generated by Claude Code