Skip to content

Commit 7a76ca8

Browse files
cleanup
1 parent 1858314 commit 7a76ca8

1 file changed

Lines changed: 1 addition & 8 deletions

File tree

  • .claude/skills/agent-device-pr-media

.claude/skills/agent-device-pr-media/SKILL.md

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ No interactive prompts. Missing inputs that cannot be defaulted hard-fail.
4444

4545
## Tests parsing rules
4646

47-
The **only hard rule**: tests live under the `### Tests` heading. Everything inside it varies wildly across authors - single numbered list, multiple numbered lists with restarts, h4 sub-headers, prose paragraphs, "N/A", or empty. Do not mechanically parse it; let the LLM interpret structure.
47+
The **only hard rule**: tests live under the `### Tests` heading. Everything inside it varies wildly across authors.
4848

4949
Operating on the `### Tests` block of the PR body:
5050

@@ -117,12 +117,6 @@ For each flow, in order:
117117
4. **On Phase 2 replay failure** (cache hit path only): the cached script is stale (UI changed under it). Delete the cache entry, mark the flow `cache: "invalidated"`, re-run Phase 1 fresh, retry Phase 2 once. If the retry still fails, mark `phase2_failed`.
118118
5. **On Phase 2 success** (cache hit path): bump `last_used_ts` and increment `hits` in the meta file.
119119

120-
### TTL / disk hygiene
121-
122-
- No hard TTL. Self-healing on replay failure handles correctness; old unused entries just sit on disk.
123-
- Soft cap: at the start of each run, prune entries with `last_used_ts` older than 30 days. Keeps the cache directory bounded without forcing regeneration of frequently-used flows.
124-
- `--cache-clear` wipes the whole cache up-front (escape hatch).
125-
126120
## Capture loop (per flow per platform)
127121

128122
Two phases per flow. Lifecycle delegated to the parent skill's bring-up. Phase 1 is skipped on cache hit (see above).
@@ -329,7 +323,6 @@ Hitting any cap marks the flow `phase1_failed` / `phase2_failed` and proceeds to
329323
- Mobile web (`iOS: mWeb Safari`, `Android: mWeb Chrome`) and Desktop (`MacOS: Chrome / Safari`) - belong in `playwright-app-testing` or a future browser-driver skill.
330324
- Standalone (non-HybridApp) builds - parent skill is HybridApp-only and this specialization inherits the gate. Production mobile evidence runs against HybridApp.
331325
- Device lifecycle (Metro, simulator boot, bundle ID resolution, session reuse, app install verification) - fully delegated to the parent skill's [Bring-up](../agent-device/SKILL.md#bring-up). This skill does not call `agent-device metro prepare`, `xcrun simctl`, or `is-hybrid-app.sh` directly.
332-
- S3 / R2 / GitHub artifact upload - deferred to Melvin (the eventual CI caller).
333326
- Editing the PR body or posting PR comments - the skill only writes local files.
334327
- Interactive prompts of any kind. CI is the eventual host; the skill must run end-to-end without human input.
335328
- Test data cleanup. Accounts/expenses/workspaces created during runs accumulate; rely on periodic test-account reset.

0 commit comments

Comments
 (0)