Skip to content

Feat/analytics platform scoped - #502

Open
michael-on-code wants to merge 17 commits into
mainfrom
feat/analytics-platform-scoped
Open

michael-on-code wants to merge 17 commits into
mainfrom
feat/analytics-platform-scoped

Conversation

@michael-on-code

Copy link
Copy Markdown
Contributor

Checklist

  • Tests were added/updated according to the feature/bugfix/change made
  • Version was rolled according to semver requirements
  • API endpoints openapi schema was updated if applicable

Changes

Feat/analytics platform scoped

Preview

image

michael-on-code and others added 13 commits August 13, 2026 08:58
…pes, layout coverage)

- vitest.config.ts: exclude `.claude/**` so test files inside Claude Code git
  worktrees (`.claude/worktrees/*`, git-ignored) are no longer collected into
  this checkout's run. Two review worktrees were adding ~9.7k failing tests
  from other branches to `pnpm test` and the coverage check. A guard test in
  __tests__/vitest-config.test.ts pins the exclude list.
- __tests__/sentry-client-config.test.ts: give the replayIntegration mock an
  options parameter so `mock.calls[0][0]` typechecks (TS2493 on tsc --noEmit).
- analytics layout test: exercise onTabChange -> router.push for a named tab
  and the overview root, lifting the mentor-scoped layout from 86.67% to 100%
  line coverage so scripts/check-test-coverage.sh passes.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@michael-on-code michael-on-code added priority run-tests Trigger PR validation pipeline labels Sep 10, 2026
@github-actions

github-actions Bot commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

PR E2E - skipped

result skipped
central run 35665058944
commit 317710d2
suite mentor - mode full - env stg1
tested against prod release 1.221.0 (env was synced to it first)

No test report was produced (result: skipped). The run did not finish a suite — usually because another PR's run took the shared environment, a newer commit/label superseded this one, or it was abandoned before completing. This says nothing about your code.

Re-trigger it yourself: comment /retest (or /retest <spec> to re-run a single file in minutes). /status shows the queue. (Avoid Re-run jobs — that replays the old merge ref instead of recomputing it against current main.)

@bnsoni bnsoni added run-tests Trigger PR validation pipeline and removed run-tests Trigger PR validation pipeline labels Sep 21, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR E2E - FAILED

Full report, traces and screenshots

result FAILED
central run 35858996193
commit 776f944e
suite mentor - mode full - env stg1
tested against prod release 1.221.0 (env was synced to it first)
failed 1
new failures 1

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority run-tests Trigger PR validation pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants