Test/e2e test pipeline by sonegillis1 · Pull Request #168 · iblai/os

sonegillis1 · 2026-05-03T23:31:33Z

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

- block-exotic-subdeps: prevent subdeps from using git/http/file URLs - onlyBuiltDependencies: allowlist packages permitted to run install scripts

mamigot · 2026-05-03T23:31:58Z

Claude finished @sonegillis1's task —— View job

E2E Coverage Review Bot — ✅ Complete

Analyzing PR #168 for e2e test coverage requirements...

Tasks:

Dismiss stale REQUEST_CHANGES reviews from bot (none found)
Identify what changed in this PR (no new page.tsx routes)
Run mechanical coverage check (✅ PASSED - 350 checkpoints, 100% coverage)
Read current coverage state
Apply decision logic and post GitHub review (APPROVED)

Changes Analysis:

.github/workflows/pr-e2e-tests.yml - CI workflow changes
CHANGELOG.md - Documentation updates
components/file-upload.tsx - Component changes
e2e/playwright.config.ts - E2E configuration changes

Coverage Check Results:

✅ All coverage checks PASSED:

Coverage unchanged: 350 checkpoints maintained
Journey coverage: 24/24 files mapped (100%)
No new app/**/page.tsx routes introduced
All existing coverage intact

Decision:

CASE E - No new page.tsx files and mechanical check passed.

✅ PR APPROVED - No e2e coverage issues detected.

mamigot

✅ E2E Coverage — No Issues

No new app routes were introduced. All existing coverage is intact.

mamigot · 2026-05-03T23:49:50Z

Claude finished @sonegillis1's task —— View job

E2E Coverage Review Bot — ✅ Complete

Analyzing PR #168 for e2e test coverage requirements...

Tasks:

Dismiss stale REQUEST_CHANGES reviews from bot (none found)
Identify what changed in this PR (no new page.tsx routes)
Run mechanical coverage check (✅ PASSED - 350 checkpoints, 100% coverage)
Read current coverage state
Apply decision logic and post GitHub review (APPROVED)

Changes Analysis:

.github/workflows/pr-e2e-tests.yml - CI workflow changes
CHANGELOG.md - Documentation updates
components/file-upload.tsx - Component changes
e2e/playwright.config.ts - E2E configuration changes

Coverage Check Results:

✅ All coverage checks PASSED:

Coverage unchanged: 350 checkpoints maintained
Journey coverage: 24/24 files mapped (100%)
No new app/**/page.tsx routes introduced
All existing coverage intact

Decision:

CASE E - No new page.tsx files and mechanical check passed.

✅ PR APPROVED - No e2e coverage issues detected.

mamigot

✅ E2E Coverage — No Issues

No new app routes were introduced. All existing coverage is intact.

mamigot · 2026-05-03T23:53:19Z

Claude finished @sonegillis1's task —— View job

E2E Coverage Review Bot — ✅ Complete

Analyzing PR #168 for e2e test coverage requirements...

Tasks:

Dismiss stale REQUEST_CHANGES reviews from bot (none found)
Identify what changed in this PR (no new page.tsx routes)
Run mechanical coverage check (✅ PASSED - 350 checkpoints, 100% coverage)
Read current coverage state
Apply decision logic and post GitHub review (APPROVED)

Changes Analysis:

CHANGELOG.md - Documentation updates
components/file-upload.tsx - Component changes
e2e/playwright.config.ts - E2E configuration changes

Coverage Check Results:

✅ All coverage checks PASSED:

Coverage unchanged: 350 checkpoints maintained
Journey coverage: 24/24 files mapped (100%)
No new app/**/page.tsx routes introduced
All existing coverage intact

Decision:

CASE E - No new page.tsx files and mechanical check passed.

✅ PR APPROVED - No e2e coverage issues detected.

mamigot

✅ E2E Coverage — No Issues

No new app routes were introduced. All existing coverage is intact.

mamigot · 2026-05-04T06:38:51Z

Claude finished @sonegillis1's task —— View job

E2E Coverage Review Bot — ✅ Complete

Analyzing PR #168 for e2e test coverage requirements...

Tasks:

Dismiss stale REQUEST_CHANGES reviews from bot (none found)
Identify what changed in this PR (no new page.tsx routes)
Run mechanical coverage check (✅ PASSED - 350 checkpoints, 100% coverage)
Read current coverage state
Apply decision logic and post GitHub review (APPROVED)

Changes Analysis:

CHANGELOG.md - Documentation updates
components/file-upload.tsx - Component changes
e2e/playwright.config.ts - E2E configuration changes

Coverage Check Results:

✅ All coverage checks PASSED:

Coverage unchanged: 350 checkpoints maintained
Journey coverage: 24/24 files mapped (100%)
No new app/**/page.tsx routes introduced
All existing coverage intact

Decision:

CASE E - No new page.tsx files and mechanical check passed.

✅ PR APPROVED - No e2e coverage issues detected.

mamigot

✅ E2E Coverage — No Issues

No new app routes were introduced. All existing coverage is intact.

mamigot · 2026-05-04T06:42:23Z

Claude finished @sonegillis1's task —— View job

E2E Coverage Review Bot — ✅ Complete

Analyzing PR #168 for e2e test coverage requirements...

Tasks:

Dismiss stale REQUEST_CHANGES reviews from bot (none found)
Identify what changed in this PR (no new page.tsx routes)
Run mechanical coverage check (✅ PASSED - 350 checkpoints, 100% coverage)
Read current coverage state
Apply decision logic and post GitHub review (APPROVED)

Changes Analysis:

.github/workflows/pr-e2e-tests.yml - CI workflow changes
CHANGELOG.md - Documentation updates
components/file-upload.tsx - Component changes
e2e/playwright.config.ts - E2E configuration changes

Coverage Check Results:

✅ All coverage checks PASSED:

Coverage unchanged: 350 checkpoints maintained
Journey coverage: 24/24 files mapped (100%)
No new app/**/page.tsx routes introduced
All existing coverage intact

Decision:

CASE E - No new page.tsx files and mechanical check passed.

✅ PR APPROVED - No e2e coverage issues detected.

mamigot

✅ E2E Coverage — No Issues

No new app routes were introduced. All existing coverage is intact.

mamigot · 2026-05-07T07:47:58Z

Claude finished @sonegillis1's task —— View job

E2E Coverage Review Bot — ✅ Complete

Analyzing PR #168 for e2e test coverage requirements...

Tasks:

Dismiss stale REQUEST_CHANGES reviews from bot (none found)
Identify what changed in this PR (no new page.tsx routes)
Run mechanical coverage check (✅ PASSED - 368 checkpoints, 100% coverage)
Read current coverage state
Apply decision logic and post GitHub review (APPROVED)

Changes Analysis:

components/file-upload.tsx - Component changes
e2e/playwright.config.ts - E2E configuration changes

Coverage Check Results:

✅ All coverage checks PASSED:

Coverage unchanged: 368 checkpoints maintained
Journey coverage: 24/24 files mapped (100%)
No new app/**/page.tsx routes introduced
All existing coverage intact

Decision:

CASE E - No new page.tsx files and mechanical check passed.

✅ PR APPROVED - No e2e coverage issues detected.

mamigot · 2026-05-07T15:56:47Z

Claude finished @sonegillis1's task —— View job

E2E Coverage Review Bot — ✅ Complete

Analyzing PR #168 for e2e test coverage requirements...

Tasks:

Dismiss stale REQUEST_CHANGES reviews from bot (none found)
Identify what changed in this PR (no new page.tsx routes)
Run mechanical coverage check (✅ PASSED - 368 checkpoints, 100% coverage)
Read current coverage state
Apply decision logic and post GitHub review (APPROVED)

Changes Analysis:

.github/workflows/pr-e2e-tests.yml - CI workflow changes
CHANGELOG.md - Documentation updates
components/file-upload.tsx - Component changes
e2e/playwright.config.ts - E2E configuration changes
yalc.lock - Package lock file

Coverage Check Results:

✅ All coverage checks PASSED:

Coverage unchanged: 368 checkpoints maintained
Journey coverage: 24/24 files mapped (100%)
No new app/**/page.tsx routes introduced
All existing coverage intact

Decision:

CASE E - No new page.tsx files and mechanical check passed.

✅ PR APPROVED - No e2e coverage issues detected.

mamigot

✅ E2E Coverage — No Issues

No new app routes were introduced. All existing coverage is intact.

mamigot · 2026-05-07T16:13:52Z

Claude finished @sonegillis1's task —— View job

E2E Coverage Review Bot — ✅ Complete

Analyzing PR #168 for e2e test coverage requirements...

Tasks:

Dismiss stale REQUEST_CHANGES reviews from bot (none found)
Identify what changed in this PR (no new page.tsx routes)
Run mechanical coverage check (✅ PASSED - 368 checkpoints, 100% coverage)
Read current coverage state
Apply decision logic and post GitHub review (APPROVED)

Changes Analysis:

.github/workflows/pr-e2e-tests.yml - CI workflow changes
components/file-upload.tsx - Component changes
e2e/playwright.config.ts - E2E configuration changes
yalc.lock - Package lock file

Coverage Check Results:

✅ All coverage checks PASSED:

Coverage unchanged: 368 checkpoints maintained
Journey coverage: 24/24 files mapped (100%)
No new app/**/page.tsx routes introduced
All existing coverage intact

Decision:

CASE E - No new page.tsx files and mechanical check passed.

✅ PR APPROVED - No e2e coverage issues detected.

mamigot

✅ E2E Coverage — No Issues

No new app routes were introduced. All existing coverage is intact.

mamigot · 2026-05-07T16:33:11Z

Claude finished @sonegillis1's task —— View job

E2E Coverage Review Bot — ✅ Complete

Analyzing PR #168 for e2e test coverage requirements...

Tasks:

Dismiss stale REQUEST_CHANGES reviews from bot (none found)
Identify what changed in this PR (no new page.tsx routes)
Run mechanical coverage check (✅ PASSED - 368 checkpoints, 100% coverage)
Read current coverage state
Apply decision logic and post GitHub review (APPROVED)

Changes Analysis:

.github/workflows/pr-e2e-tests.yml - CI workflow changes
components/file-upload.tsx - Component changes
e2e/playwright.config.ts - E2E configuration changes
yalc.lock - Package lock file

Coverage Check Results:

✅ All coverage checks PASSED:

Coverage unchanged: 368 checkpoints maintained
Journey coverage: 24/24 files mapped (100%)
No new app/**/page.tsx routes introduced
All existing coverage intact

Decision:

CASE E - No new page.tsx files and mechanical check passed.

✅ PR APPROVED - No e2e coverage issues detected.

mamigot

✅ E2E Coverage — No Issues

No new app routes were introduced. All existing coverage is intact.

mamigot · 2026-05-07T16:43:39Z

Claude finished @sonegillis1's task —— View job

E2E Coverage Review Bot — ✅ Complete

Analyzing PR #168 for e2e test coverage requirements...

Tasks:

Dismiss stale REQUEST_CHANGES reviews from bot (none found)
Identify what changed in this PR (no new page.tsx routes)
Run mechanical coverage check (✅ PASSED - 368 checkpoints, 100% coverage)
Read current coverage state
Apply decision logic and post GitHub review (APPROVED)

Changes Analysis:

.github/workflows/pr-e2e-tests.yml - CI workflow changes
components/file-upload.tsx - Component changes
e2e/playwright.config.ts - E2E configuration changes
yalc.lock - Package lock file

Coverage Check Results:

✅ All coverage checks PASSED:

Coverage unchanged: 368 checkpoints maintained
Journey coverage: 24/24 files mapped (100%)
No new app/**/page.tsx routes introduced
All existing coverage intact

Decision:

CASE E - No new page.tsx files and mechanical check passed.

✅ PR APPROVED - No e2e coverage issues detected.

mamigot

✅ E2E Coverage — No Issues

No new app routes were introduced. All existing coverage is intact.

…tils - Dockerfile: override package.json version via APP_VERSION build arg - pr-e2e: use wildcard sed pattern for image replacement - Add LTI test utilities and update canvas embed spec

mamigot · 2026-05-08T08:34:59Z

Claude finished @sonegillis1's task —— View job

E2E Coverage Review Bot — ✅ Complete

Analyzing PR #168 for e2e test coverage requirements...

Tasks:

Dismiss stale REQUEST_CHANGES reviews from bot (none found)
Identify what changed in this PR (no new page.tsx routes)
Run mechanical coverage check (✅ PASSED - 368 checkpoints, 100% coverage)
Read current coverage state
Apply decision logic and post GitHub review (APPROVED)

Changes Analysis:

.github/workflows/pr-e2e-tests.yml - CI workflow changes
Dockerfile - Docker configuration changes
components/file-upload.tsx - Component changes
e2e/journeys/11-canvas-embed.spec.ts - E2E journey updates
e2e/playwright.config.ts - E2E configuration changes
e2e/utils/auth.ts - E2E auth utilities
e2e/utils/lti.ts - New LTI utilities
yalc.lock - Package lock file

Coverage Check Results:

✅ All coverage checks PASSED:

Coverage unchanged: 368 checkpoints maintained
Journey coverage: 24/24 files mapped (100%)
No new app/**/page.tsx routes introduced
All existing coverage intact

Decision:

CASE E - No new page.tsx files and mechanical check passed.

✅ PR APPROVED - No e2e coverage issues detected.

mamigot

✅ E2E Coverage — No Issues

No new app routes were introduced. All existing coverage is intact.

mamigot · 2026-05-08T08:54:50Z

Claude finished @sonegillis1's task —— View job

E2E Coverage Review Bot — ✅ Complete

Analyzing PR #168 for e2e test coverage requirements...

Tasks:

Dismiss stale REQUEST_CHANGES reviews from bot (none found)
Identify what changed in this PR (no new page.tsx routes)
Run mechanical coverage check (✅ PASSED - 368 checkpoints, 100% coverage)
Read current coverage state
Apply decision logic and post GitHub review (APPROVED)

Changes Analysis:

.github/workflows/pr-e2e-tests.yml - CI workflow changes
Dockerfile - Docker configuration changes
components/file-upload.tsx - Component changes
e2e/journeys/11-canvas-embed.spec.ts - E2E journey updates
e2e/playwright.config.ts - E2E configuration changes
e2e/utils/auth.ts - E2E auth utilities
e2e/utils/lti.ts - New LTI utilities
yalc.lock - Package lock file

Coverage Check Results:

✅ All coverage checks PASSED:

Coverage unchanged: 368 checkpoints maintained
Journey coverage: 24/24 files mapped (100%)
No new app/**/page.tsx routes introduced
All existing coverage intact

Decision:

CASE E - No new page.tsx files and mechanical check passed.

✅ PR APPROVED - No e2e coverage issues detected.

mamigot

✅ E2E Coverage — No Issues

No new app routes were introduced. All existing coverage is intact.

- Fix journey 11 to traverse xblock iframe + shadow DOM correctly - Add CANVAS_PATH_WITH_LTI_MENTOR env var for specific LMS page - Add globalSetup for dotenv loading in Playwright workers - Fix useOS hook to use useState/useEffect (avoid SSR issues) - Add w-fit to version badge, items-center to card content - Remove debug console.logs from playwright config and test-data

mamigot · 2026-05-08T10:58:09Z

Claude finished @sonegillis1's task —— View job

E2E Coverage Review Bot — ✅ Complete

Analyzing PR #168 for e2e test coverage requirements...

Tasks:

Dismiss stale REQUEST_CHANGES reviews from bot (none found)
Identify what changed in this PR (no new page.tsx routes)
Run mechanical coverage check (✅ PASSED - 368 checkpoints, 100% coverage)
Read current coverage state
Apply decision logic and post GitHub review (APPROVED)

Changes Analysis:

.github/workflows/pr-e2e-tests.yml - CI workflow changes
Dockerfile - Docker configuration changes
components/file-upload.tsx - Component changes
e2e/journeys/11-canvas-embed.spec.ts - E2E journey updates
e2e/playwright.config.ts - E2E configuration changes
e2e/utils/auth.ts - E2E auth utilities
e2e/utils/lti.ts - New LTI utilities
yalc.lock - Package lock file

Coverage Check Results:

✅ All coverage checks PASSED:

Coverage unchanged: 368 checkpoints maintained
Journey coverage: 24/24 files mapped (100%)
No new app/**/page.tsx routes introduced
All existing coverage intact

Decision:

CASE E - No new page.tsx files and mechanical check passed.

✅ PR APPROVED - No e2e coverage issues detected.

mamigot

✅ E2E Coverage — No Issues

No new app routes were introduced. All existing coverage is intact.

mamigot · 2026-05-08T13:00:40Z

Claude finished @sonegillis1's task —— View job

E2E Coverage Review Bot — ✅ Complete

Analyzing PR #168 for e2e test coverage requirements...

Tasks:

Dismiss stale REQUEST_CHANGES reviews from bot (none found)
Identify what changed in this PR (no new page.tsx routes)
Run mechanical coverage check (✅ PASSED - 368 checkpoints, 100% coverage)
Read current coverage state
Apply decision logic and post GitHub review (APPROVED)

Changes Analysis:

.github/workflows/pr-e2e-tests.yml - CI workflow changes
Dockerfile - Docker configuration changes
components/file-upload.tsx - Component changes
e2e/journeys/11-canvas-embed.spec.ts - E2E journey updates
e2e/playwright.config.ts - E2E configuration changes
e2e/utils/auth.ts - E2E auth utilities
e2e/utils/lti.ts - New LTI utilities
yalc.lock - Package lock file

Coverage Check Results:

✅ All coverage checks PASSED:

Coverage unchanged: 368 checkpoints maintained
Journey coverage: 24/24 files mapped (100%)
No new app/**/page.tsx routes introduced
All existing coverage intact

Decision:

CASE E - No new page.tsx files and mechanical check passed.

✅ PR APPROVED - No e2e coverage issues detected.

mamigot

✅ E2E Coverage — No Issues

No new app routes were introduced. All existing coverage is intact.

sonegillis1 added 7 commits April 30, 2026 21:16

test: updates

48e37ee

chore: retrigger pipeline

9d4485c

chore(security): add pnpm supply chain protections

035480c

- block-exotic-subdeps: prevent subdeps from using git/http/file URLs - onlyBuiltDependencies: allowlist packages permitted to run install scripts

Merge branch 'main' into test/e2e-test-pipeline

1b6dd43

fix(ci): remove trailing commas in pr-e2e-tests YAML

d8a3774

ci: fix pr-e2e-tests

b13593b

Merge branch 'main' into test/e2e-test-pipeline

40d5c3b

sonegillis1 requested a review from mamigot as a code owner May 3, 2026 23:31

mamigot approved these changes May 3, 2026

View reviewed changes

sonegillis1 added the run-tests Trigger PR validation pipeline label May 3, 2026

chore: merge main

621b56e

mamigot approved these changes May 3, 2026

View reviewed changes

Merge branch 'main' into test/e2e-test-pipeline

a530e3f

mamigot approved these changes May 3, 2026

View reviewed changes

sonegillis1 closed this May 4, 2026

sonegillis1 reopened this May 4, 2026

mamigot approved these changes May 4, 2026

View reviewed changes

ci(pr-e2e): add unlabeled trigger to allow re-running via label toggle

00dae72

mamigot approved these changes May 4, 2026

View reviewed changes

sonegillis1 added run-tests Trigger PR validation pipeline and removed run-tests Trigger PR validation pipeline labels May 4, 2026

Merge branch 'main' into test/e2e-test-pipeline

c9b49c6

ci: resolve merge conflicts

99a7db3

mamigot approved these changes May 7, 2026

View reviewed changes

Merge branch 'main' into test/e2e-test-pipeline

17db150

mamigot approved these changes May 7, 2026

View reviewed changes

sonegillis1 added run-tests Trigger PR validation pipeline and removed run-tests Trigger PR validation pipeline labels May 7, 2026

Merge branch 'main' into test/e2e-test-pipeline

8621786

mamigot approved these changes May 7, 2026

View reviewed changes

Merge branch 'main' into test/e2e-test-pipeline

ddb2a36

mamigot approved these changes May 7, 2026

View reviewed changes

sonegillis1 added run-tests Trigger PR validation pipeline and removed run-tests Trigger PR validation pipeline labels May 7, 2026

ci: add APP_VERSION build arg, update pr-e2e workflow, and LTI test u…

3f5bb16

…tils - Dockerfile: override package.json version via APP_VERSION build arg - pr-e2e: use wildcard sed pattern for image replacement - Add LTI test utilities and update canvas embed spec

mamigot approved these changes May 8, 2026

View reviewed changes

ci(pr-e2e): remove explicit image-tag, let reusable workflow default

4449522

mamigot approved these changes May 8, 2026

View reviewed changes

Merge branch 'main' into test/e2e-test-pipeline

ed56bf8

mamigot approved these changes May 8, 2026

View reviewed changes

Conversation

sonegillis1 commented May 3, 2026

Checklist

Changes

Uh oh!

mamigot commented May 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

E2E Coverage Review Bot — ✅ Complete

Tasks:

Changes Analysis:

Coverage Check Results:

Decision:

Uh oh!

mamigot left a comment

Choose a reason for hiding this comment

✅ E2E Coverage — No Issues

Uh oh!

mamigot commented May 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

E2E Coverage Review Bot — ✅ Complete

Tasks:

Changes Analysis:

Coverage Check Results:

Decision:

Uh oh!

mamigot left a comment

Choose a reason for hiding this comment

✅ E2E Coverage — No Issues

Uh oh!

mamigot commented May 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

E2E Coverage Review Bot — ✅ Complete

Tasks:

Changes Analysis:

Coverage Check Results:

Decision:

Uh oh!

mamigot left a comment

Choose a reason for hiding this comment

✅ E2E Coverage — No Issues

Uh oh!

mamigot commented May 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

E2E Coverage Review Bot — ✅ Complete

Tasks:

Changes Analysis:

Coverage Check Results:

Decision:

Uh oh!

mamigot left a comment

Choose a reason for hiding this comment

✅ E2E Coverage — No Issues

Uh oh!

mamigot commented May 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

E2E Coverage Review Bot — ✅ Complete

Tasks:

Changes Analysis:

Coverage Check Results:

Decision:

Uh oh!

mamigot left a comment

Choose a reason for hiding this comment

✅ E2E Coverage — No Issues

Uh oh!

mamigot commented May 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

E2E Coverage Review Bot — ✅ Complete

Tasks:

Changes Analysis:

Coverage Check Results:

Decision:

Uh oh!

mamigot commented May 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

E2E Coverage Review Bot — ✅ Complete

Tasks:

Changes Analysis:

Coverage Check Results:

Decision:

mamigot commented May 3, 2026 •

edited

Loading

mamigot commented May 3, 2026 •

edited

Loading

mamigot commented May 3, 2026 •

edited

Loading

mamigot commented May 4, 2026 •

edited

Loading

mamigot commented May 4, 2026 •

edited

Loading

mamigot commented May 7, 2026 •

edited

Loading

mamigot commented May 7, 2026 •

edited

Loading

mamigot commented May 7, 2026 •

edited

Loading

mamigot commented May 7, 2026 •

edited

Loading

mamigot commented May 7, 2026 •

edited

Loading

mamigot commented May 8, 2026 •

edited

Loading

mamigot commented May 8, 2026 •

edited

Loading