@codyswann/lisa 2.195.4 → 2.195.6

package/plugins/lisa-expo/.codex-plugin/skills/ops-verify-health/SKILL.md

@@ -0,0 +1,101 @@
+---
+name: ops-verify-health
+description: "Health check all services…"
+allowed-tools:
+  - Bash
+  - Read
+---
+
+# Ops: Verify Health
+
+Health check all services across environments.
+
+**Argument**: `$ARGUMENTS` — environment(s) to check (default: `all`). Options: `local`, `dev`, `staging`, `production`, `all`
+
+## Discovery
+
+Read these files to build the environment URL table:
+
+1. `e2e/constants.ts` — frontend URLs per environment
+2. `.env.localhost`, `.env.development`, `.env.staging`, `.env.production` — `EXPO_PUBLIC_GRAPHQL_BASE_URL` for backend GraphQL URLs
+
+## Health Checks
+
+For each environment, run these checks:
+
+### Frontend Check
+
+```bash
+curl -sf -o /dev/null -w "HTTP %{http_code} in %{time_total}s" {frontend_url}
+```
+
+Verify the response contains Expo/React markers:
+```bash
+curl -sf {frontend_url} | head -20
+```
+
+### Backend GraphQL Check
+
+```bash
+curl -sf {graphql_url} -X POST \
+  -H "Content-Type: application/json" \
+  -d '{"query":"{ __typename }"}' \
+  -w "\nHTTP %{http_code} in %{time_total}s\n"
+```
+
+### GraphQL Introspection (detailed check)
+
+```bash
+curl -sf {graphql_url} -X POST \
+  -H "Content-Type: application/json" \
+  -d '{"query":"{ __schema { queryType { name } } }"}' \
+  -w "\nHTTP %{http_code} in %{time_total}s\n"
+```
+
+## Full Health Check Script
+
+Run all checks for the specified environment(s):
+
+```bash
+check_env() {
+  local ENV=$1
+  local FE_URL=$2
+  local BE_URL=$3
+
+  echo "=== $ENV ==="
+
+  # Frontend
+  FE_STATUS=$(curl -sf -o /dev/null -w "%{http_code}" --max-time 10 "$FE_URL" 2>/dev/null || echo "000")
+  FE_TIME=$(curl -sf -o /dev/null -w "%{time_total}" --max-time 10 "$FE_URL" 2>/dev/null || echo "timeout")
+  echo "Frontend: HTTP $FE_STATUS ($FE_TIME s)"
+
+  # Backend
+  BE_RESULT=$(curl -sf --max-time 10 "$BE_URL" -X POST \
+    -H "Content-Type: application/json" \
+    -d '{"query":"{ __typename }"}' \
+    -w "\n%{http_code} %{time_total}" 2>/dev/null || echo -e "\n000 timeout")
+  BE_STATUS=$(echo "$BE_RESULT" | tail -1 | awk '{print $1}')
+  BE_TIME=$(echo "$BE_RESULT" | tail -1 | awk '{print $2}')
+  echo "Backend:  HTTP $BE_STATUS ($BE_TIME s)"
+  echo ""
+}
+```
+
+## EAS Update Status
+
+Check the latest OTA updates deployed to each branch:
+
+```bash
+eas update:list --branch {env} --limit 3
+```
+
+## Output Format
+
+Report results as a table:
+
+| Service | Environment | Status | Response Time | Details |
+|---------|-------------|--------|---------------|---------|
+| Frontend | dev | UP (200) | 0.45s | HTML contains React root |
+| Backend | dev | UP (200) | 0.32s | GraphQL responds `__typename` |
+
+Flag any service with status != 200 or response time > 5s as a concern.

package/plugins/lisa-expo/.codex-plugin/skills/ops-verify-health/agents/openai.yaml

@@ -0,0 +1,4 @@
+display_name: "Ops Verify Health"
+short_description: "Health check all services…"
+default_prompt:
+  - "Use $ops-verify-health: Health check all services…."

package/plugins/lisa-expo/.codex-plugin/skills/owasp-zap/SKILL.md

@@ -0,0 +1,61 @@
+---
+name: owasp-zap
+description: "running or interpreting an…"
+---
+
+# OWASP ZAP Baseline Scanning
+
+OWASP ZAP (Zed Attack Proxy) performs DAST (Dynamic Application Security Testing) by scanning a running application for common security vulnerabilities from the OWASP Top 10.
+
+## When to Use
+
+- After making changes to HTTP headers, authentication, or security middleware
+- Before deploying to staging or production
+- When reviewing security scan results from CI
+- When triaging ZAP findings from pull request checks
+
+## Running Locally
+
+```bash
+# Requires Docker to be installed and running
+bash scripts/zap-baseline.sh
+```
+
+The scan builds the Expo web export, serves it locally, and runs ZAP against it. Reports are saved to `zap-report.html`, `zap-report.json`, and `zap-report.md`.
+
+## Interpreting Results
+
+ZAP findings are categorized by risk level:
+
+| Risk | Action |
+|------|--------|
+| **High** | Fix immediately — indicates exploitable vulnerability |
+| **Medium** | Fix before deployment — security best practice violation |
+| **Low** | Fix when convenient — minor security improvement |
+| **Informational** | Review — may be false positive or acceptable risk |
+
+## Common Findings and Fixes
+
+### Infrastructure-Level (fix at CDN/hosting, not in code)
+
+- **CSP Header Not Set**: Configure Content-Security-Policy at CDN or hosting platform. Expo web exports need `script-src 'self' 'unsafe-inline'` for hydration.
+- **HSTS Not Set**: Configure Strict-Transport-Security at CDN/load balancer.
+- **X-Frame-Options**: Use `frame-ancestors` in CSP at CDN level.
+
+### Application-Level (fix in code)
+
+- **Cookie flags missing**: Ensure all cookies set `HttpOnly`, `Secure`, and `SameSite` attributes.
+- **Debug error messages**: Ensure error boundaries don't leak stack traces in production.
+- **Server version disclosure**: Remove or mask the `Server` response header.
+
+## Configuration
+
+ZAP scan rules are configured in `.zap/baseline.conf`. Each line controls how ZAP treats a specific rule:
+
+- `IGNORE`: Skip the rule entirely
+- `WARN`: Report finding but don't fail the build
+- `FAIL`: Fail the build if this finding is detected
+
+## CI Integration
+
+ZAP runs automatically in CI via the `zap-baseline.yml` workflow. Results are uploaded as artifacts and the build fails on medium+ severity findings.

package/plugins/lisa-expo/.codex-plugin/skills/owasp-zap/agents/openai.yaml

@@ -0,0 +1,4 @@
+display_name: "OWASP ZAP"
+short_description: "running or interpreting an…"
+default_prompt:
+  - "Use $owasp-zap: running or interpreting an…."

package/plugins/lisa-expo/.codex-plugin/skills/play-store-access/SKILL.md

@@ -0,0 +1,226 @@
+---
+name: play-store-access
+description: "Read-only Google Play Console…"
+version: 1.0.0
+license: MIT
+allowed-tools:
+  - Bash
+  - Read
+---
+
+# Play Store Access: $ARGUMENTS
+
+Read-only access layer for Google Play release state in Expo projects. Use it to
+answer whether the Android app is live, halted, draft, or in staged rollout on a
+Play track after EAS submission.
+
+This skill MUST NOT promote tracks, change rollout percentages, create releases,
+or submit builds. EAS remains the submission path.
+
+## Invocation Contract
+
+```text
+operation: setup
+operation: production-status [track:production] [package:<android.package>]
+operation: track-status track:<track> [package:<android.package>]
+operation: resolve-package
+```
+
+Return a concise result with the package name, track, release `name` /
+`versionCodes`, `status`, `userFraction` when present, and any `countryTargeting`
+or in-app update priority fields present in the API response.
+
+## Configuration
+
+Project-safe config can live in `.lisa.config.json`:
+
+```json
+{
+  "playStore": {
+    "packageName": "com.example.app",
+    "track": "production"
+  }
+}
+```
+
+Local credential config belongs in `.lisa.config.local.json`, which must stay
+gitignored:
+
+```json
+{
+  "playStore": {
+    "serviceAccountKeyPath": "./google-play-service-account.json"
+  }
+}
+```
+
+Supported credential locations, in resolution order:
+
+1. `GOOGLE_PLAY_SERVICE_ACCOUNT_JSON` containing the raw service-account JSON.
+2. `GOOGLE_PLAY_SERVICE_ACCOUNT_KEY_BASE64` containing base64 JSON.
+3. `.lisa.config.local.json` `playStore.serviceAccountKeyPath`.
+4. Darwin keychain item `lisa-google-play` account `<packageName>`.
+
+Fail closed if no credential is available. Never prompt indefinitely and never
+fall back to Play Console UI scraping.
+
+## One-Time Setup Flow
+
+For `operation: setup`:
+
+1. Confirm the project is an Expo app by checking for `app.json`,
+   `app.config.json`, `app.config.js`, or `app.config.ts`.
+2. Resolve or ask the operator to provide the Android package name. Prefer
+   `.lisa.config.json` `playStore.packageName`, then `expo.android.package` from
+   app config.
+3. In Google Cloud Console, create a service account dedicated to read-only Play
+   release visibility and create a JSON key for it.
+4. In Play Console, open **Setup -> API access**, link the Google Cloud project if
+   needed, then grant the service-account email app access with read-level
+   release visibility. Do not grant release-management write permissions for v1.
+5. Store the key locally using one of the credential locations above. If using a
+   path, add the path to `.gitignore` and record only the path in
+   `.lisa.config.local.json`.
+6. Record non-secret defaults in `.lisa.config.json`:
+
+   ```bash
+   jq '.playStore.packageName = "com.example.app" | .playStore.track = "production"' \
+     .lisa.config.json > /tmp/lisa-config.json && mv /tmp/lisa-config.json .lisa.config.json
+   ```
+
+The key material itself must never be committed.
+
+## Resolve Package Name
+
+Resolution order:
+
+1. Explicit `package:<value>` argument.
+2. `.lisa.config.json` `playStore.packageName`.
+3. `app.json` / `app.config.json` `expo.android.package`.
+4. `app.config.ts` / `app.config.js` by running Expo config export:
+
+   ```bash
+   npx expo config --json | jq -r '.android.package // empty'
+   ```
+
+If the package still cannot be resolved, fail with:
+
+```text
+Error: Google Play package name is not configured. Set playStore.packageName in .lisa.config.json or pass package:<name>.
+```
+
+## Read Track State
+
+The androidpublisher API requires an edit even for track reads. Create the edit,
+list the track, then delete the edit. Do not commit the edit.
+
+```bash
+PACKAGE_NAME="${PACKAGE_NAME:-$(jq -r '.playStore.packageName // empty' .lisa.config.json)}"
+TRACK="${TRACK:-$(jq -r '.playStore.track // "production"' .lisa.config.json)}"
+KEY_PATH="$(jq -r '.playStore.serviceAccountKeyPath // empty' .lisa.config.local.json 2>/dev/null)"
+
+read_service_account_json() {
+  if [ -n "${GOOGLE_PLAY_SERVICE_ACCOUNT_JSON:-}" ]; then
+    printf '%s' "$GOOGLE_PLAY_SERVICE_ACCOUNT_JSON"
+    return
+  fi
+  if [ -n "${GOOGLE_PLAY_SERVICE_ACCOUNT_KEY_BASE64:-}" ]; then
+    printf '%s' "$GOOGLE_PLAY_SERVICE_ACCOUNT_KEY_BASE64" | base64 --decode 2>/dev/null ||
+      printf '%s' "$GOOGLE_PLAY_SERVICE_ACCOUNT_KEY_BASE64" | base64 -D
+    return
+  fi
+  if [ -n "$KEY_PATH" ] && [ -f "$KEY_PATH" ]; then
+    cat "$KEY_PATH"
+    return
+  fi
+  if [ "$(uname -s)" = "Darwin" ] && [ -n "$PACKAGE_NAME" ]; then
+    security find-generic-password -s lisa-google-play -a "$PACKAGE_NAME" -w 2>/dev/null
+    return
+  fi
+}
+
+SERVICE_ACCOUNT_JSON="$(read_service_account_json)"
+[ -n "$SERVICE_ACCOUNT_JSON" ] || {
+  echo "Error: no Google Play service-account key found. Run play-store-access operation: setup." >&2
+  exit 1
+}
+[ -n "$PACKAGE_NAME" ] || {
+  echo "Error: Google Play package name is not configured. Set playStore.packageName or pass package:<name>." >&2
+  exit 1
+}
+
+ACCESS_TOKEN="$(
+  SERVICE_ACCOUNT_JSON="$SERVICE_ACCOUNT_JSON" node <<'NODE'
+const crypto = require("crypto");
+
+const account = JSON.parse(process.env.SERVICE_ACCOUNT_JSON);
+const now = Math.floor(Date.now() / 1000);
+const encode = (value) =>
+  Buffer.from(JSON.stringify(value)).toString("base64url");
+const unsigned = `${encode({ alg: "RS256", typ: "JWT" })}.${encode({
+  iss: account.client_email,
+  scope: "https://www.googleapis.com/auth/androidpublisher",
+  aud: "https://oauth2.googleapis.com/token",
+  iat: now,
+  exp: now + 3600,
+})}`;
+const signature = crypto
+  .createSign("RSA-SHA256")
+  .update(unsigned)
+  .sign(account.private_key, "base64url");
+
+fetch("https://oauth2.googleapis.com/token", {
+  method: "POST",
+  headers: { "Content-Type": "application/x-www-form-urlencoded" },
+  body: new URLSearchParams({
+    grant_type: "urn:ietf:params:oauth:grant-type:jwt-bearer",
+    assertion: `${unsigned}.${signature}`,
+  }),
+})
+  .then(async (response) => {
+    const body = await response.json();
+    if (!response.ok) {
+      throw new Error(JSON.stringify(body));
+    }
+    process.stdout.write(body.access_token);
+  })
+  .catch((error) => {
+    console.error(`Error: failed to obtain Google Play access token: ${error.message}`);
+    process.exit(1);
+  });
+NODE
+)"
+
+BASE="https://androidpublisher.googleapis.com/androidpublisher/v3/applications/${PACKAGE_NAME}"
+EDIT_ID="$(curl -fsS -X POST "${BASE}/edits" -H "Authorization: Bearer ${ACCESS_TOKEN}" | jq -r '.id')"
+trap 'curl -fsS -X DELETE "${BASE}/edits/${EDIT_ID}" -H "Authorization: Bearer ${ACCESS_TOKEN}" >/dev/null 2>&1 || true' EXIT
+
+curl -fsS "${BASE}/edits/${EDIT_ID}/tracks/${TRACK}" \
+  -H "Authorization: Bearer ${ACCESS_TOKEN}" |
+  jq --arg packageName "$PACKAGE_NAME" \
+    '{packageName: $packageName, track: .track, releases: (.releases // [])}'
+```
+
+Interpretation:
+
+- `status: completed` means the release is live on that track.
+- `status: inProgress` means staged rollout; inspect `userFraction`.
+- `status: halted` means rollout was stopped.
+- `status: draft` means it is not live.
+- Absence of a production release means the app has no watchable production-track
+  release through the API.
+
+Known limitation: androidpublisher exposes track/release state well, but
+pre-publication review state is coarse. The reliable watchable signal is whether
+the target version is live on the production track.
+
+## Error Handling
+
+- `401` / token exchange failure: service-account key is invalid or expired.
+- `403`: service account is not invited to the Play Console app or lacks read
+  access to releases.
+- `404`: package name is wrong, the app is not created in Play Console, or the
+  service account has no access to that app.
+
+In all cases, report the package name and credential source used, but never print
+private key material or the access token.

package/plugins/lisa-expo/.codex-plugin/skills/play-store-access/agents/openai.yaml

@@ -0,0 +1,4 @@
+display_name: "Play Store Access"
+short_description: "Read-only Google Play Console…"
+default_prompt:
+  - "Use $play-store-access: Read-only Google Play Console…."

package/plugins/lisa-expo/.codex-plugin/skills/playwright-ci-debugging/SKILL.md

@@ -0,0 +1,140 @@
+---
+name: playwright-ci-debugging
+description: "Debug Playwright E2E tests that…"
+---
+
+# Debugging Playwright E2E Failures in CI
+
+The authoring-side rules (selectors, testID forwarding, naming) are in the `playwright-selectors` skill. This skill is for the other half of the job: when a test fails in CI and you need to find out why.
+
+## Debugging Order
+
+When a Playwright E2E test fails in CI, follow this exact order.
+
+### 1. Reproduce locally first — before anything else
+
+Start the project's dev server, then run the failing test in isolation:
+
+```bash
+# Start the dev server (discover the script from package.json — commonly `start`, `dev`, `start:dev`, or `web`)
+<pkg-manager> run <dev-script>
+
+# In another terminal, run the exact failing test with no retries
+BASE_URL=http://localhost:8081/ npx playwright test <file> --grep "<test name>" --retries=0
+```
+
+`--retries=0` is critical — retries mask flake. `--grep` isolates the single failing test so you're not waiting on a full suite.
+
+**Do NOT read source code, CI logs, or theorize until you can reproduce locally.** Most CI failures reproduce locally if you run the same test against the same served build. Guessing from logs is slow and usually wrong.
+
+### 2. Intercept the network request
+
+When a test involves an API call, set up a Playwright response listener and inspect the status code and response body. A 400/500 response tells you everything.
+
+```typescript
+page.on("response", async (response) => {
+  if (response.url().includes("/api/")) {
+    console.log(response.status(), response.url(), await response.text());
+  }
+});
+```
+
+UI failures are often downstream of a failed API call. The network log is cheaper evidence than the DOM.
+
+### 3. If it doesn't reproduce locally, understand the CI environment first
+
+CI runs against a different environment than your laptop. Before changing anything:
+
+- Find the CI job that runs Playwright (usually `.github/workflows/*.yml`).
+- Identify which `.env.*` file it loads — this varies by target branch (e.g., dev → `.env.development`, staging → `.env.staging`).
+- Note that CI typically builds a static web bundle (`expo export --platform web`) then serves it on `localhost:8081` via `serve dist`. It is NOT hitting your dev server. Timing, bundling, and env vars all differ.
+- Reproduce the CI setup locally: build the static bundle, serve it the same way, then run the test. If it now fails locally, you've isolated an env/build difference.
+
+### 4. Verify commit SHA before trusting CI results
+
+After pushing, confirm the CI run's `headSha` matches your latest commit:
+
+```bash
+gh run list --branch "$(git branch --show-current)" --limit 1 --json headSha,status,conclusion
+git rev-parse HEAD
+```
+
+Bots (review-response, auto-update, dependabot) can push commits between your push and the CI run, overwriting your changes. A green check on a stale SHA tells you nothing about your fix.
+
+## Patterns That Eliminate Flake
+
+### Never use fixed waits before interactions
+
+`waitForTimeout()` as the sole wait before a click is a silent failure waiting to happen — animations and rendering take variable time, especially on slower CI runners. Poll for the expected state, then act.
+
+```typescript
+// BAD — fixed wait; click silently fails if element not yet visible
+await clickVisibleText(page, "Translate");
+await page.waitForTimeout(1000);
+await clickVisibleText(page, "Spanish");
+
+// GOOD — poll for visibility, then click
+await clickVisibleText(page, "Translate");
+await expect
+  .poll(() => hasVisibleText(page, "Spanish"), { timeout: TIMEOUT.expect })
+  .toBe(true);
+await clickVisibleText(page, "Spanish");
+```
+
+### Never silently swallow click return values
+
+Any helper that can return `false` on a missed click (e.g., `clickVisibleText`) should either have its return value asserted or be preceded by a visibility poll. A click that silently returns `false` is a hidden test bug — the test proceeds as if the click happened and fails downstream with a confusing error.
+
+```typescript
+// BAD — return value ignored; test continues on failed click
+await clickVisibleText(page, "Submit");
+
+// GOOD — assert the click happened
+expect(await clickVisibleText(page, "Submit")).toBe(true);
+
+// GOOD — precede with visibility poll
+await expect
+  .poll(() => hasVisibleText(page, "Submit"), { timeout: TIMEOUT.expect })
+  .toBe(true);
+await clickVisibleText(page, "Submit");
+```
+
+### E2E tests must not depend on external API success
+
+Any test that calls an external service (AWS Bedrock, third-party APIs, rate-limited providers) must handle the failure case. Tests should verify UI behavior, not external service uptime. If an external call might fail, the test must accept both outcomes or skip the dependent assertion.
+
+```typescript
+// BAD — assumes translation always succeeds
+await expect
+  .poll(() => hasVisibleText(page, "Show Original"), { timeout: TIMEOUT.expect })
+  .toBe(true);
+
+// GOOD — handle both success and failure
+await expect
+  .poll(
+    async () =>
+      (await hasVisibleText(page, "Show Original")) ||
+      (await hasVisibleText(page, "Translate")),
+    { timeout: TIMEOUT.expect }
+  )
+  .toBe(true);
+
+const translated = await hasVisibleText(page, "Show Original");
+if (!translated) return; // External API failed; skip downstream assertions
+```
+
+The alternative — mocking the external call — is a valid approach when the goal is to test the UI's handling of a successful response. Pick one strategy per test and commit to it.
+
+### E2E assertions must not depend on specific test data
+
+Do not assert specific text (e.g., "No lists detected") when the test user's data state varies across environments. If a zero-row state could have different empty-state messages depending on the user's data, either check for multiple possible states or skip the assertion entirely.
+
+See the `playwright-selectors` skill's "Data independence" section for authoring patterns that avoid this class of bug in the first place.
+
+## Escalation
+
+If you've worked through steps 1–4 and still cannot explain the failure:
+
+- Do NOT disable, skip, or `.fixme` the test to unblock the PR.
+- Do NOT use `--admin` or force-merge to bypass the check.
+- Escalate to a human with: the failing test name, your local reproduction attempt, the network trace, and the commit SHA verification. A real flake that cannot be root-caused is a legitimate reason to pause; it is never a reason to merge red.

package/plugins/lisa-expo/.codex-plugin/skills/playwright-ci-debugging/agents/openai.yaml

@@ -0,0 +1,4 @@
+display_name: "Playwright CI Debugging"
+short_description: "Debug Playwright E2E tests that…"
+default_prompt:
+  - "Use $playwright-ci-debugging: Debug Playwright E2E tests that…."