Release/2.0.21#2226
Merged
Merged
Conversation
Adds a new @packrat/env workspace package with a Node-runtime shim that parses process.env through a Zod schema once at startup, caches the result, and throws an aggregated error listing every missing or invalid variable. Node/Bun scripts import env.FOO (or requireNodeEnv(['FOO'])) instead of reaching into process.env.FOO directly, so missing required vars fail loudly instead of silently returning undefined later in the pipeline. Migrates three high-value callers as proof of the pattern: - packages/api/migrate.ts (NEON_DATABASE_URL) - packages/api/src/db/seed.ts (NEON_DATABASE_URL) - packages/analytics/scripts/smoke-test.ts (R2_* credentials) Out of scope for this PR: Expo client (EXPO_PUBLIC_*), Next.js landing/guides, and the Cloudflare Worker API which already has its own context-aware getEnv(c) shim at packages/api/src/utils/env-validation.ts. The preinstall hook .github/scripts/configure-deps.ts is intentionally not migrated because it runs before node_modules symlinks exist.
Agent-Logs-Url: https://github.com/PackRat-AI/PackRat/sessions/64974b12-aef2-4934-b7ad-7bc55240b1b5 Co-authored-by: andrew-bierman <94939237+andrew-bierman@users.noreply.github.com>
- Rebase onto development (resolve analytics/package.json conflict: keep @packrat/env dep + use catalog: for chalk) - Fix @ts-ignore → @ts-expect-error in useCreatePackFromPack.ts - Add comment-stripping to no-circular-deps.ts (prevents JSDoc false positives) - Break circular imports: catalog/types, profile/types, itemCalculations now point to specific type files instead of barrel indexes - Create packs/input.ts and api/types/validation.ts to break dep cycles - Fix schema.ts to import ValidationError from types/validation - Convert apps/expo and packages/env deps to catalog: (bun catalog entries added by #2070 — ai, axios, radash, react, react-dom, tailwindcss, ts, zod) - Sort package.json files
…shim # Conflicts: # apps/expo/lib/utils/itemCalculations.ts Co-authored-by: andrew-bierman <94939237+andrew-bierman@users.noreply.github.com>
Agent-Logs-Url: https://github.com/PackRat-AI/PackRat/sessions/82555f7b-9ebc-448f-b6cf-f66d136d627c Co-authored-by: andrew-bierman <94939237+andrew-bierman@users.noreply.github.com>
…, and API container Agent-Logs-Url: https://github.com/PackRat-AI/PackRat/sessions/0363de50-f658-4a76-9379-9de74ff9800f Co-authored-by: andrew-bierman <94939237+andrew-bierman@users.noreply.github.com>
Adds four new endpoints under /api/admin/analytics (Basic Auth protected): - GET /analytics — Chart.js HTML dashboard with period selector - GET /analytics/growth — Time-series: user registrations, pack creation, catalog additions - GET /analytics/activity — Time-series: trips, trail condition reports, social posts - GET /analytics/breakdown — Categorical: packs by category, pack items by category All endpoints support period (day/week/month) and range query params. Adds Analytics nav link to the existing admin panel layout. https://claude.ai/code/session_017iNQZKZ9tbC8AfJ6qL1aqu
Packages:
- Rename packages/analytics → packages/data-lake (@packrat/data-lake)
Pure data-lake library: DuckDB/Iceberg cache managers, query builder,
enrichment, entity resolver — no CLI deps
- Create packages/cli (@packrat/cli)
Extracts CLI entry + all 23 commands from data-lake; imports
@packrat/data-lake as a workspace dep. Room to add non-catalog CLI
commands without coupling them to the data-lake library.
- Update all @packrat/analytics imports → @packrat/data-lake in tests,
tsconfigs, and cross-package references
Admin analytics API (packages/api/src/routes/admin/analytics/):
- platform.ts — renamed from analytics.ts; routes unchanged under /platform/*
- catalog.ts — new: 5 PostgreSQL-backed endpoints for the gear catalog
GET /catalog/overview total items, brands, pricing, availability,
embedding coverage, items added last 30 days
GET /catalog/brands top brands by count with avg/min/max price
GET /catalog/prices price distribution across 6 buckets
GET /catalog/etl ETL pipeline history + summary stats
GET /catalog/embeddings pgvector embedding coverage %
- index.ts — mounts /platform + /catalog sub-routers; serves updated
HTML dashboard with separate "Platform" and "Gear Catalog"
sections, Chart.js charts for all 13 data sources
All admin analytics routes remain Basic Auth protected.
https://claude.ai/code/session_017iNQZKZ9tbC8AfJ6qL1aqu
Full-featured admin dashboard in apps/admin following the same Next.js 15 / shadcn / Tailwind patterns as apps/landing and apps/guides. Key features: - Collapsible shadcn sidebar with Dashboard, Platform Analytics, Gear Catalog Analytics, Users, Packs, Catalog, and Settings nav - Dashboard overview: stat cards + area chart (growth), availability breakdown, and embedding coverage progress bar - Platform analytics page: line + bar charts for growth/activity (period selector: day/week/month) and pack category donut chart - Gear catalog page: price distribution (horizontal bar), availability donut, top brands bar chart, embedding coverage, and ETL job table with success-rate badges - React Query v5 data fetching via AdminAPI client (lib/api.ts) - All charts use the shadcn ChartContainer / recharts wrappers - `bun admin` script added to root package.json - admin-app/* path alias added to root tsconfig.json - .env.local.example documents NEXT_PUBLIC_API_URL and Basic Auth vars https://claude.ai/code/session_017iNQZKZ9tbC8AfJ6qL1aqu
Two chronic failure modes on `development` were: 1. iOS run wasted ~32 min building the sim app only to fail at the Maestro step when `E2E_TEST_EMAIL` / `E2E_TEST_PASSWORD` secrets were empty. Fail fast with an actionable message before any expensive work runs. 2. Android Gradle OOM during `:app:packageRelease` on `ubuntu-latest`. Free disk, add a 10G swap file, cap parallelism, and bump the heap so R8/packaging fits. Also set `_JAVA_OPTIONS` + the namespaced `ORG_GRADLE_PROJECT_*` env so the heap bump reaches Gradle even when launched indirectly by eas.
Lets us validate the Android Gradle heap/swap fix without waiting for the E2E_TEST_EMAIL/E2E_TEST_PASSWORD secrets to be populated. When secrets are missing the build runs to completion and the Maestro/ simulator/emulator steps are skipped via a step-level if guard.
Removes the dependency on a manually-created preview DB user. Adds `packages/api/src/db/seed-e2e-user.ts` which upserts the E2E account (creating if missing, refreshing password hash + emailVerified flag if present) using NEON_DEV_DATABASE_URL. The workflow now: - Hardcodes TEST_EMAIL to admin+e2e-test-automation@packratai.com - Requires only E2E_TEST_PASSWORD + NEON_DEV_DATABASE_URL secrets (the latter already exists) - Runs the seed step on both iOS and Android jobs immediately before Maestro, so the test account always matches the password CI expects
Both TEST_EMAIL and TEST_PASSWORD now come from repo secrets, and the seed script already reads its inputs purely from env vars — no hard- coded identities anywhere. Pre-flight widened to enumerate exactly which of E2E_TEST_EMAIL / E2E_TEST_PASSWORD / NEON_DEV_DATABASE_URL is missing.
Drops the duplicate bcrypt import — if the app ever swaps the hashing algorithm or salt rounds, the seed stays in sync automatically.
- seed: drop returning({id}) arg (drizzle infers); use non-! destructure
- swap: ubuntu-latest already has /swapfile mounted, use /mnt/swapfile-ci
and dd instead of fallocate (which reports 'Text file busy')
Removed NDK, Python, go, temurin-jdk, azul jdks from the prune list —
the RN Gradle build links against the Android SDK/NDK installed on
the runner ("NDK was not found under …/sdk/ndk/27.3.13750724"), and
Python is load-bearing for some RN build scripts.
Previously we downgraded the secret preflight to a warning so the Android Gradle heap/swap fix could be validated without the secrets being populated. That validation is now done (PR #2176's prior run built the APK cleanly in 35m42s) and the lenient mode produced a misleading green check: every Maestro/emulator step was skipped via if-guards but the job conclusion was still 'success'. Revert to the original behavior: hard-fail with a clear error listing which of E2E_TEST_EMAIL / E2E_TEST_PASSWORD / NEON_DEV_DATABASE_URL is missing, and drop every `has_secrets == 'true'` guard now that the job fails on the very first step when those secrets aren't set.
iOS Maestro run failed with "iOS driver not ready in time" because the xcuitest install/boot on a fresh macos-15 runner takes longer than the 180s default. Set MAESTRO_DRIVER_STARTUP_TIMEOUT=600000ms.
- Refactor getPayloadConfigFromPayload to accept 2 params (opts object) instead of 3 separate params, matching the guides app pattern and satisfying biome's complexity/useMaxParams rule (max: 2) - Sort packages/data-lake/package.json per monorepo conventions https://claude.ai/code/session_017iNQZKZ9tbC8AfJ6qL1aqu
The no-duplicate-deps lint script checks peerDependencies too. >=2.15.0 in web-ui conflicted with the pinned 3.8.1 in admin/landing/guides, causing the biome CI step to exit 1. https://claude.ai/code/session_017iNQZKZ9tbC8AfJ6qL1aqu
Apply biome 2.4.6 --write fixes to all files affected by the merge from main: organizeImports, useTemplate, noUnusedVariables, and other auto-fixable rules. Also addresses stale suppressions in chart.tsx. https://claude.ai/code/session_017iNQZKZ9tbC8AfJ6qL1aqu
The EAS e2e profile builds the preview variant (app.config.ts appends '.preview' to the bundle id when IS_PREVIEW). Maestro was launching com.andrewbierman.packrat but the installed app is com.andrewbierman.packrat.preview, causing clear-state to fail with: Failed to get app binary directory for bundle ... No such file or directory Matches what .maestro/README.md already documents.
Prevents Next.js build cache from being tracked as untracked files. https://claude.ai/code/session_017iNQZKZ9tbC8AfJ6qL1aqu
Caches that were missing: - root `node_modules` on both jobs (keyed by bun.lock) - iOS CocoaPods download cache (keyed by package.json + app.config.ts) Expected savings on warm hits: - bun install: ~40s → <5s (both jobs) - pod install: ~3-5 min → <1 min (iOS) Still dominated by xcodebuild/gradle compilation which is CPU-bound on free-tier runners — can't cache those without a paid build cache service or a bigger runner.
reactivecircus/android-emulator-runner@v2.37 runs each line of the
`script:` argument as a separate `sh -c`, which breaks our backslash
line continuations — `maestro test \` was executed by itself,
parsing `\` as the flow path ("Flow path does not exist").
Keep it single-line.
Inline `maestro test` didn't pass `-e APP_ID=...` so flows saw
`${APP_ID}` → undefined → 'Unable to launch app undefined'.
e2e.sh already forwards TEST_EMAIL/TEST_PASSWORD/APP_ID/TRIP_NAME/
PACK_NAME plus date-window vars the same way iOS does.
Contributor
|
Important Review skippedToo many files! This PR contains 156 files, which is 6 over the limit of 150. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (156)
You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
github-actions
Bot
added
documentation
Contributor
Coverage Report for API Unit Tests Coverage (./packages/api)
File CoverageNo changed files found. |
Contributor
Coverage Report for Expo Unit Tests Coverage (./apps/expo)
File Coverage
|
||||||||||||||||||||||||||||||||||||||||||||||||||
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the monorepo’s CI/workflows and adds several reliability/maintainability improvements across tooling and packages (typed env shims, shared config/constants, a shared API client, new admin analytics endpoints/UI), alongside version bumps for the 2.0.21 release.
Changes:
- Consolidates CI checks into a unified
checks.ymlworkflow and strengthens E2E workflow preparation/secret validation. - Introduces shared packages/utilities (
@packrat/env,@packrat/config,@packrat/api-client,@packrat/checks) and new repo lint/check scripts. - Adds Admin analytics API routes and UI components; updates MCP tools to use the shared API client and enum-based schemas.
Reviewed changes
Copilot reviewed 154 out of 157 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| tsconfig.json | Updates root TS compiler options, adds workspace path aliases, and adjusts includes/excludes. |
| scripts/lint/check-react-doctor.ts | Adds a monorepo-wide “doctor:react” runner across apps. |
| scripts/check-all.ts | Adds new checks (magic strings + react doctor) to the consolidated check runner. |
| packages/web-ui/src/components/chart.tsx | Refines chart config handling and displayName values; adds Biome ignore for controlled dangerouslySetInnerHTML. |
| packages/web-ui/package.json | Moves recharts to dev+peer deps and marks it optional. |
| packages/ui/package.json | Bumps package version. |
| packages/mcp/vitest.config.ts | Adds Vitest alias for @packrat/api-client to use workspace source. |
| packages/mcp/tsconfig.json | Adds baseUrl/paths for local package imports. |
| packages/mcp/src/tools/weather.ts | Adjusts API call typing and switches season suggestions endpoint method. |
| packages/mcp/src/tools/trips.ts | Changes list trips tool schema/params to include_public. |
| packages/mcp/src/tools/trail-conditions.ts | Replaces literal enums with shared nativeEnum definitions. |
| packages/mcp/src/tools/packs.ts | Switches schemas to shared enums; changes update/remove endpoints/methods. |
| packages/mcp/src/tools/catalog.ts | Switches to page-based paging + enum sort params; tightens item fetch typing. |
| packages/mcp/src/resources.ts | Refactors resource error helper to a single options object. |
| packages/mcp/src/prompts.ts | Switches prompt schemas to shared enums. |
| packages/mcp/src/index.ts | Uses shared API client factory; centralizes routes/service metadata constants. |
| packages/mcp/src/enums.ts | Adds shared enums for MCP tool schemas. |
| packages/mcp/src/constants.ts | Adds constants for MCP worker routes and service metadata. |
| packages/mcp/src/client.ts | Re-exports API client primitives from @packrat/api-client. |
| packages/mcp/src/tests/tools/weather.test.ts | Updates tests for new ApiError signature and POST season suggestions. |
| packages/mcp/src/tests/tools/trips.test.ts | Updates tests for new trips list schema/params and ApiError signature. |
| packages/mcp/src/tests/tools/trail-conditions.test.ts | Updates tests for new ApiError signature. |
| packages/mcp/src/tests/tools/packs.test.ts | Updates tests for endpoint/method changes and ApiError signature. |
| packages/mcp/src/tests/tools/knowledge.test.ts | Updates tests for new ApiError signature. |
| packages/mcp/src/tests/tools/catalog.test.ts | Updates tests for page-based paging and ApiError signature. |
| packages/mcp/src/tests/client.test.ts | Updates tests for new ApiError constructor options. |
| packages/mcp/src/tests/auth.test.ts | Improves McpAgent mock shape to include instance fetch. |
| packages/mcp/package.json | Bumps version and adds dependency on @packrat/api-client. |
| packages/guards/package.json | Bumps version. |
| packages/env/src/node.ts | Adds Node/Bun typed env shim (nodeEnv). |
| packages/env/src/next.ts | Adds Next.js typed env shim (guideEnv). |
| packages/env/src/index.ts | Exposes Node env shim at package root and type exports. |
| packages/env/src/expo-server.ts | Adds Expo server-only env shim. |
| packages/env/src/expo-client.ts | Adds Expo client env shim with explicit process.env reads for Metro. |
| packages/env/scripts/no-raw-process-env.ts | Adds lint script to flag raw process.env usage outside allowlist. |
| packages/env/package.json | Introduces new @packrat/env package. |
| packages/config/src/index.ts | Adds entrypoint export for config constants. |
| packages/config/src/config.ts | Adds shared, deeply-frozen app config (flags + dashboard layout/strings). |
| packages/config/package.json | Introduces new @packrat/config package. |
| packages/cli/src/index.ts | Derives CLI version from package.json and adds robust top-level error handling. |
| packages/cli/src/commands/trends.ts | Uses shared arg parsing helpers. |
| packages/cli/src/commands/specs.ts | Uses shared arg parsing helpers and cache initializer. |
| packages/cli/src/commands/search.ts | Uses shared arg parsing helpers. |
| packages/cli/src/commands/sales.ts | Uses shared arg parsing helpers. |
| packages/cli/src/commands/reviews.ts | Uses shared arg parsing helpers. |
| packages/cli/src/commands/resolve.ts | Uses shared arg parsing helpers. |
| packages/cli/src/commands/ratings.ts | Uses shared arg parsing helpers. |
| packages/cli/src/commands/market-share.ts | Uses shared arg parsing helpers. |
| packages/cli/src/commands/lightweight.ts | Uses shared arg parsing helpers. |
| packages/cli/src/commands/images.ts | Uses shared arg parsing helpers. |
| packages/cli/src/commands/filter.ts | Uses shared arg parsing helpers and cache initializer. |
| packages/cli/src/commands/export.ts | Uses shared arg parsing helpers. |
| packages/cli/src/commands/deals.ts | Uses shared arg parsing helpers. |
| packages/cli/src/commands/compare.ts | Uses shared arg parsing helpers. |
| packages/cli/src/commands/category.ts | Uses shared arg parsing helpers. |
| packages/cli/src/commands/build-specs.ts | Uses cache initializer consistently. |
| packages/cli/src/commands/brands.ts | Uses shared arg parsing helpers. |
| packages/cli/src/commands/brand.ts | Uses shared arg parsing helpers. |
| packages/cli/src/args.ts | Adds centralized Zod-based CLI arg parsers. |
| packages/cli/scripts/smoke-test.ts | Migrates S3/R2 env reads to nodeEnv. |
| packages/cli/package.json | Adds bin entry and depends on @packrat/env + zod; bumps version. |
| packages/checks/src/check-magic-strings.ts | Adds a repo scan for repeated “magic strings”. |
| packages/checks/package.json | Introduces new @packrat/checks package. |
| packages/api/test/upload.test.ts | Switches to using seeded+logged-in test user helper. |
| packages/api/test/setup.ts | Tightens mock typing for batchVectorSearch. |
| packages/api/test/pack-templates.test.ts | Switches to using seeded+logged-in test user helper. |
| packages/api/test/image-detection.test.ts | Switches to using seeded+logged-in test user helper. |
| packages/api/src/routes/index.ts | Exports AppRoutes type for typed API client usage. |
| packages/api/src/routes/guides/getGuidesRoute.ts | Extracts regex patterns into constants for guide id/title derivation. |
| packages/api/src/routes/ai/index.ts | Validates RAG search response via schema before returning. |
| packages/api/src/routes/admin/index.ts | Wires new analytics routes into admin router and adds nav link. |
| packages/api/src/routes/admin/analytics/platform.ts | Adds platform analytics endpoints (growth/activity/breakdown). |
| packages/api/src/routes/admin/analytics/index.ts | Adds analytics sub-router aggregation and index response. |
| packages/api/src/routes/admin/analytics/catalog.ts | Adds catalog analytics endpoints (overview/brands/prices/etl/embeddings). |
| packages/api/src/db/seed.ts | Uses nodeEnv for NEON DB URL. |
| packages/api/src/db/seed-e2e-user.ts | Adds E2E user upsert seed script for CI. |
| packages/api/package.json | Adds E2E seed script and depends on @packrat/env; bumps version. |
| packages/api/migrate.ts | Uses nodeEnv for NEON DB URL. |
| packages/api/container_src/server.ts | Improves env validation and defers Google AI client init until env is validated. |
| packages/api/container_src/package.json | Bumps version. |
| packages/api/TESTING.md | Updates CI workflow references to consolidated checks workflow. |
| packages/api-client/tsconfig.json | Adds TypeScript config for the new API client package. |
| packages/api-client/src/index.ts | Adds shared HTTP client + error + MCP result helpers. |
| packages/api-client/package.json | Introduces new @packrat/api-client package. |
| packages/analytics/test/integration/local-mode.test.ts | Migrates S3 creds check to nodeEnv. |
| packages/analytics/test/integration/connection.test.ts | Migrates S3/catalog creds checks to nodeEnv. |
| packages/analytics/test/integration/catalog-mode.test.ts | Migrates catalog creds checks to nodeEnv. |
| packages/analytics/src/core/spec-parser.ts | Extracts common regex patterns/constants for maintainability. |
| packages/analytics/src/core/query-builder.ts | Adds Biome ignore for static-only class namespace usage. |
| packages/analytics/src/core/entity-resolver.ts | Extracts common regex constants and avoids repeated inline patterns. |
| packages/analytics/src/core/enrichment.ts | Extracts repeated regex patterns and normalizes URL cleanup steps. |
| packages/analytics/src/core/data-export.ts | Extracts file extension regex constant for summary file writing. |
| packages/analytics/src/core/catalog-cache.ts | Removes unused DuckDBInstance reference, simplifies catalog connection state. |
| packages/analytics/package.json | Depends on @packrat/env; bumps version. |
| package.json | Adds new check scripts and extends lint:custom with no-raw-process-env; bumps version. |
| copilot-instructions.md | Updates workflow references and documents new Copilot setup workflow. |
| biome.json | Adds MCP index file to ignore list. |
| apps/landing/package.json | Adds doctor:react script; bumps version. |
| apps/guides/scripts/update-authors.ts | Refactors loops away from forEach; adjusts exports ordering. |
| apps/guides/package.json | Adds doctor:react script and @packrat/env; bumps version. |
| apps/guides/lib/enhanceGuideContent.ts | Uses guideEnv instead of raw process.env. |
| apps/guides/app/dev/generate/page.tsx | Uses guideEnv.NODE_ENV instead of raw process.env. |
| apps/guides/app/api/dev/generate-post/route.ts | Uses guideEnv.NODE_ENV instead of raw process.env. |
| apps/guides/app/api/dev/generate-batch/route.ts | Uses guideEnv.NODE_ENV instead of raw process.env. |
| apps/expo/package.json | Adds doctor:react, depends on config/env, bumps Expo + adds deps; bumps version. |
| apps/expo/lib/utils/buildPackTemplateItemImageUrl.ts | Migrates env import to @packrat/env/expo-client. |
| apps/expo/lib/utils/buildImageUrl.ts | Migrates env import to @packrat/env/expo-client. |
| apps/expo/lib/utils/tests/buildPackTemplateItemImageUrl.test.ts | Updates mock path for new env module. |
| apps/expo/lib/utils/tests/buildImageUrl.test.ts | Updates mock path for new env module. |
| apps/expo/lib/api/client.ts | Migrates env import to @packrat/env/expo-client. |
| apps/expo/features/trips/screens/TripWeatherDetailsScreen.tsx | Replaces any with typed weather response and improves rendering safety. |
| apps/expo/features/trips/screens/TripDetailScreen.tsx | Removes unused weather locations hook/state. |
| apps/expo/features/feed/utils/index.ts | Migrates env import to @packrat/env/expo-client. |
| apps/expo/features/feed/utils/tests/feedUtils.test.ts | Updates mock path for new env module. |
| apps/expo/features/catalog/components/CatalogItemsAuthWall.tsx | Replaces header component usage with Expo Router screen config. |
| apps/expo/features/auth/hooks/useAuthInit.ts | Migrates env import to @packrat/env/expo-client. |
| apps/expo/features/auth/hooks/useAuthActions.ts | Migrates env import to @packrat/env/expo-client. |
| apps/expo/features/ai/components/AIModeSelector.tsx | Gates UI behind feature flag. |
| apps/expo/env/serverEnvs.ts | Removes old Expo server env shim (replaced by @packrat/env/expo-server). |
| apps/expo/env/clientEnvs.ts | Removes old Expo client env shim (replaced by @packrat/env/expo-client). |
| apps/expo/eas.json | Adjusts Gradle/Java memory tuning for Android builds. |
| apps/expo/config.ts | Replaces inline flags with shared @packrat/config. |
| apps/expo/app/_layout.tsx | Uses typed Expo client env for Sentry DSN. |
| apps/expo/app/(app)/trip/location-search.tsx | Uses typed Expo client env for Google Maps key fallback. |
| apps/expo/app/(app)/ai-chat.tsx | Gates local model init/transport checks behind feature flag. |
| apps/expo/app/(app)/(tabs)/profile/index.tsx | Migrates env import to @packrat/env/expo-client. |
| apps/expo/app/(app)/(tabs)/(home)/index.tsx | Centralizes dashboard layout + strings via shared config and reduces “gap” magic strings. |
| apps/expo/app.config.ts | Bumps app version. |
| apps/admin/package.json | Adds doctor:react script and bumps version. |
| apps/admin/lib/api.ts | Adds typed API wrappers for new analytics endpoints. |
| apps/admin/hooks/use-platform-analytics.ts | Adds React Query hooks for platform analytics endpoints. |
| apps/admin/hooks/use-catalog-analytics.ts | Adds React Query hooks for catalog analytics endpoints. |
| apps/admin/config/nav.ts | Adds analytics navigation entries. |
| apps/admin/components/page-header.tsx | Adds shared page header component for admin pages. |
| apps/admin/components/dashboard/dashboard-content.tsx | Adds dashboard analytics widgets + charts. |
| apps/admin/components/analytics/platform-analytics.tsx | Adds platform analytics page content (charts + period selection). |
| apps/admin/components/analytics/catalog-analytics.tsx | Adds catalog analytics page content (charts + ETL table). |
| apps/admin/app/users/page.tsx | Adds placeholder users page with header. |
| apps/admin/app/settings/page.tsx | Adds placeholder settings page with header. |
| apps/admin/app/packs/page.tsx | Adds placeholder packs page with header. |
| apps/admin/app/dashboard/analytics/platform/page.tsx | Adds platform analytics route page. |
| apps/admin/app/dashboard/analytics/catalog/page.tsx | Adds catalog analytics route page. |
| apps/admin/app/catalog/page.tsx | Adds placeholder catalog page with header. |
| README.md | Replaces old workflow badges with consolidated checks badge. |
| .gitignore | Adds .next and additional local/worktree ignores. |
| .github/workflows/unit-tests.yml | Uses frozen Bun install. |
| .github/workflows/sync-guides-r2.yml | Uses frozen Bun install. |
| .github/workflows/migrations.yml | Uses frozen Bun install. |
| .github/workflows/e2e-tests.yml | Adds secret validation, caching, seed step, and reliability improvements for E2E. |
| .github/workflows/copilot-setup-steps.yml | Adds robust Copilot agent environment bootstrap workflow. |
| .github/workflows/checks.yml | Adds unified checks workflow (Biome + custom checks + typecheck). |
| .github/workflows/check-types.yml | Removes old check-types workflow (replaced by checks.yml). |
| .github/workflows/biome.yml | Removes old biome workflow (replaced by checks.yml). |
| .github/copilot-setup-steps.yml | Removes old Copilot setup steps file (replaced by workflow). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
andrew-bierman
approved these changes
Apr 18, 2026
Collaborator
Author
|
@copilot apply changes based on the comments in this thread |
Agent-Logs-Url: https://github.com/PackRat-AI/PackRat/sessions/c3f60a75-a528-428d-9667-a06baabb20d3 Co-authored-by: mikib0 <54102880+mikib0@users.noreply.github.com>
Contributor
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ❌ Deployment failed View logs |
packrat-admin | d6d1165 | Apr 18 2026, 05:02 PM |
Contributor
All changes applied in d6d1165:
|
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.
This pull request refactors the GitHub Actions CI workflows to consolidate and improve checks, optimize dependency management, and enhance E2E test reliability. The changes include merging and replacing several workflows with a new unified
checks.yml, optimizing caching and install steps, and adding robust secret validation and environment setup for E2E tests.Workflow consolidation and improvements:
.github/workflows/checks.ymlto unify linting, type checking, Biome checks, and other code quality steps, replacing the previous separate workflows for Biome and type checks. This workflow also supports an autofix mode for Biome via workflow dispatch..github/workflows/biome.ymland.github/workflows/check-types.ymlas their functionality is now handled by the newchecks.ymlworkflow. [1] [2]bun install --frozen-lockfileacross all workflows to ensure lockfile consistency. [1] [2] [3] [4] [5] [6]E2E testing enhancements:
node_modulesand CocoaPods to speed up repeated runs, and updated the Android emulator runner to a newer version. [1] [2] [3]Copilot setup and validation:
.github/copilot-setup-steps.ymlwith a new, robust.github/workflows/copilot-setup-steps.ymlworkflow that validates required secrets, pins Node and Bun versions, and verifies toolchain readiness for Copilot. [1] [2]Miscellaneous improvements:
These changes streamline CI, improve reliability, and make local and cloud development environments more predictable and robust.