Skip to content

Feat/mentor/1797#264

Open
razasaddiqi wants to merge 23 commits into
mainfrom
feat/mentor/1797
Open

Feat/mentor/1797#264
razasaddiqi wants to merge 23 commits into
mainfrom
feat/mentor/1797

Conversation

@razasaddiqi

@razasaddiqi razasaddiqi commented Jun 16, 2026

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

Close iblai/iblai-platform#1797

  • Add Private mode for the chat per agent, tenant, user global level, and embed mode
  • Add new journey 50 for private mode playwright tests
  • Fix playwright tests for journey 04, 14, 34, 44, 45,
3 1 2 5 8 image image image image

@razasaddiqi razasaddiqi requested a review from mamigot as a code owner June 16, 2026 21:23
@mamigot

mamigot commented Jun 16, 2026

Copy link
Copy Markdown
Member

Claude encountered an error —— View job


I'll analyze this and get back to you.

@mamigot

mamigot commented Jun 16, 2026

Copy link
Copy Markdown
Member

Claude encountered an error —— View job


I'll analyze this and get back to you.

@mamigot

mamigot commented Jun 16, 2026

Copy link
Copy Markdown
Member

Claude encountered an error —— View job


I'll analyze this and get back to you.

@razasaddiqi razasaddiqi added the run-tests Trigger PR validation pipeline label Jun 16, 2026
@mamigot

mamigot commented Jun 17, 2026

Copy link
Copy Markdown
Member

Claude finished @razasaddiqi's task —— View job


E2E Coverage Bot

  • Dismiss stale REQUEST_CHANGES reviews (none found)
  • Identify changed files — no new page.tsx routes added
  • Run mechanical coverage check — ✅ passed (443 → 463 checkpoints, +20)
  • Read current coverage state
  • Applied decision logic: CASE E — No new routes, coverage check passed → Approved ✅

Summary

No new app routes were introduced in this PR. The coverage check passed with an improvement of +20 checkpoints (443 → 463). A new e2e journey spec (50-chat-privacy.spec.ts) and updated coverage files were included by the developer.
| Branch: feat/mentor/1797

@mamigot mamigot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ E2E Coverage — No Issues

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

Coverage check result: 443 → 463 checkpoints (+20 improvements)

@mamigot

mamigot commented Jun 17, 2026

Copy link
Copy Markdown
Member

Claude finished @razasaddiqi's task —— View job


E2E Coverage Bot

  • Dismiss stale REQUEST_CHANGES reviews (none found)
  • Identify changed files — no new page.tsx routes added
  • Run mechanical coverage check — ✅ passed (443 → 463 checkpoints, +20)
  • Read current coverage state
  • Applied decision logic: CASE E — No new routes, coverage check passed → Approved ✅

Summary

No new app routes were introduced in this PR. The coverage check passed with an improvement of +20 checkpoints (443 → 463). A new e2e journey spec (50-chat-privacy.spec.ts) and updated coverage files were included by the developer.
| Branch: feat/mentor/1797

@mamigot mamigot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ E2E Coverage — No Issues

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

Coverage check result: 443 → 463 checkpoints (+20 improvements)

@mamigot

mamigot commented Jun 17, 2026

Copy link
Copy Markdown
Member

Claude finished @razasaddiqi's task —— View job


E2E Coverage Bot

  • Dismiss stale REQUEST_CHANGES reviews (none found)
  • Identify changed files — no new page.tsx routes added
  • Run mechanical coverage check — ✅ passed (443 → 463 checkpoints, +20)
  • Read current coverage state
  • Applied decision logic: CASE E — No new routes, coverage check passed → Approved ✅

Summary

No new app routes were introduced in this PR. The coverage check passed with an improvement of +20 checkpoints (443 → 463). A new e2e journey spec (50-chat-privacy.spec.ts) and updated coverage files were included by the developer.
| Branch: feat/mentor/1797

@mamigot mamigot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ E2E Coverage — No Issues

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

Coverage check result: 443 → 463 checkpoints (+20 improvements)

@mamigot

mamigot commented Jun 17, 2026

Copy link
Copy Markdown
Member

Claude finished @razasaddiqi's task —— View job


E2E Coverage Bot

  • Dismiss stale REQUEST_CHANGES reviews (none found)
  • Identify changed files — no new page.tsx routes added
  • Run mechanical coverage check — ✅ passed (443 → 463 checkpoints, +20)
  • Read current coverage state
  • Applied decision logic: CASE E — No new routes, coverage check passed → Approved ✅

Summary

No new app routes were introduced in this PR. The coverage check passed with an improvement of +20 checkpoints (443 → 463). A new e2e journey spec (50-chat-privacy.spec.ts) and updated coverage files were included by the developer.
| Branch: feat/mentor/1797

@mamigot mamigot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ E2E Coverage — No Issues

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

Coverage check result: 443 → 463 checkpoints (+20 improvements)

@razasaddiqi razasaddiqi added run-tests Trigger PR validation pipeline and removed run-tests Trigger PR validation pipeline labels Jun 17, 2026
@mamigot

mamigot commented Jun 17, 2026

Copy link
Copy Markdown
Member

Claude finished @razasaddiqi's task —— View job


E2E Coverage Bot

  • Dismiss stale REQUEST_CHANGES reviews (none found)
  • Identify changed files — no new page.tsx routes added
  • Run mechanical coverage check — ✅ passed (443 → 463 checkpoints, +20)
  • Read current coverage state
  • Applied decision logic: CASE E — No new routes, coverage check passed → Approved ✅

Summary

No new app routes were introduced in this PR. The coverage check passed with an improvement of +20 checkpoints (443 → 463). A new e2e journey spec (50-chat-privacy.spec.ts) and updated coverage files were included by the developer.
| Branch: feat/mentor/1797

@mamigot mamigot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ E2E Coverage — No Issues

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

Coverage check result: 443 → 463 checkpoints (+20 improvements)

@mamigot

mamigot commented Jun 17, 2026

Copy link
Copy Markdown
Member

Claude finished @razasaddiqi's task —— View job


E2E Coverage Bot

  • Dismiss stale REQUEST_CHANGES reviews (none found)
  • Identify changed files — no new page.tsx routes added
  • Run mechanical coverage check — ✅ passed (443 → 463 checkpoints, +20)
  • Read current coverage state
  • Applied decision logic: CASE E — No new routes, coverage check passed → Approved ✅

Summary

No new app routes were introduced in this PR. The coverage check passed with an improvement of +20 checkpoints (443 → 463). A new e2e journey spec (50-chat-privacy.spec.ts) and updated coverage files were included by the developer.
| Branch: feat/mentor/1797

@mamigot mamigot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ E2E Coverage — No Issues

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

Coverage check result: 443 → 463 checkpoints (+20 improvements)

@mamigot

mamigot commented Jun 17, 2026

Copy link
Copy Markdown
Member

Claude finished @razasaddiqi's task —— View job


E2E Coverage Bot

  • Dismiss stale REQUEST_CHANGES reviews (none found)
  • Identify changed files — no new page.tsx routes added
  • Run mechanical coverage check — ✅ passed (443 → 463 checkpoints, +20)
  • Read current coverage state
  • Applied decision logic: CASE E — No new routes, coverage check passed → Approved ✅

Summary

No new app routes were introduced in this PR. The coverage check passed with an improvement of +20 checkpoints (443 → 463). A new e2e journey spec (50-chat-privacy.spec.ts) and updated coverage files were included by the developer.
| Branch: feat/mentor/1797

@mamigot mamigot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ E2E Coverage — No Issues

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

Coverage check result: 443 → 463 checkpoints (+20 improvements)

@mamigot

mamigot commented Jun 17, 2026

Copy link
Copy Markdown
Member

Claude finished @razasaddiqi's task —— View job


E2E Coverage Bot

  • Dismiss stale REQUEST_CHANGES reviews (none found)
  • Identify changed files — no new page.tsx routes added
  • Run mechanical coverage check — ✅ passed (443 → 463 checkpoints, +20)
  • Read current coverage state
  • Applied decision logic: CASE E — No new routes, coverage check passed → Approved ✅

Summary

No new app routes were introduced in this PR. The coverage check passed with an improvement of +20 checkpoints (443 → 463). A new e2e journey spec (50-chat-privacy.spec.ts) and updated coverage files were included by the developer.
| Branch: feat/mentor/1797

@mamigot mamigot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ E2E Coverage — No Issues

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

Coverage check result: 443 → 463 checkpoints (+20 improvements)

@mamigot

mamigot commented Jun 18, 2026

Copy link
Copy Markdown
Member

Claude finished @razasaddiqi's task —— View job


E2E Coverage Bot

  • Dismiss stale REQUEST_CHANGES reviews (none found)
  • Identify changed files — no new page.tsx routes added
  • Run mechanical coverage check — ✅ passed (443 → 463 checkpoints, +20)
  • Read current coverage state
  • Applied decision logic: CASE E — No new routes, coverage check passed → Approved ✅

Summary

No new app routes were introduced in this PR. The coverage check passed with an improvement of +20 checkpoints (443 → 463). A new e2e journey spec (50-chat-privacy.spec.ts) and updated coverage files were included by the developer.
| Branch: feat/mentor/1797

@mamigot mamigot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ E2E Coverage — No Issues

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

Coverage check result: 443 → 463 checkpoints (+20 improvements)

@mamigot

mamigot commented Jun 18, 2026

Copy link
Copy Markdown
Member

Claude finished @razasaddiqi's task —— View job


E2E Coverage Bot

  • Dismiss stale REQUEST_CHANGES reviews (none found)
  • Identify changed files — no new page.tsx routes added
  • Run mechanical coverage check — ✅ passed (443 → 463 checkpoints, +20)
  • Read current coverage state
  • Applied decision logic: CASE E — No new routes, coverage check passed → Approved ✅

Summary

No new app routes were introduced in this PR. The coverage check passed with an improvement of +20 checkpoints (443 → 463). A new e2e journey spec (50-chat-privacy.spec.ts) and updated coverage files were included by the developer.
| Branch: feat/mentor/1797

@mamigot mamigot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ E2E Coverage — No Issues

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

Coverage check result: 443 → 463 checkpoints (+20 improvements)

@mamigot

mamigot commented Jun 18, 2026

Copy link
Copy Markdown
Member

Claude finished @razasaddiqi's task —— View job


E2E Coverage Bot

  • Dismiss stale REQUEST_CHANGES reviews (none found)
  • Identify changed files — no new page.tsx routes added
  • Run mechanical coverage check — ✅ passed (448 → 468 checkpoints, +20)
  • Read current coverage state
  • Applied decision logic: CASE E — No new routes, coverage check passed → Approved ✅

Summary

No new app routes were introduced in this PR. The coverage check passed with an improvement of +20 checkpoints (448 → 468). A new e2e journey spec (50-chat-privacy.spec.ts) and updated coverage files were included by the developer.
| Branch: feat/mentor/1797

@mamigot mamigot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ E2E Coverage — No Issues

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

Coverage check result: 448 → 468 checkpoints (+20 improvements)

@mamigot

mamigot commented Jun 18, 2026

Copy link
Copy Markdown
Member

Claude finished @razasaddiqi's task —— View job


E2E Coverage Bot

  • Dismiss stale REQUEST_CHANGES reviews (none found)
  • Identify changed files — no new page.tsx routes added
  • Run mechanical coverage check — ✅ passed (448 → 468 checkpoints, +20)
  • Read current coverage state
  • Applied decision logic: CASE E — No new routes, coverage check passed → Approved ✅

Summary

No new app routes were introduced in this PR. The coverage check passed with an improvement of +20 checkpoints (448 → 468). A new e2e journey spec (50-chat-privacy.spec.ts) and updated coverage files were included by the developer.
| Branch: feat/mentor/1797

@mamigot mamigot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ E2E Coverage — No Issues

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

Coverage check result: 448 → 468 checkpoints (+20 improvements)

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

Labels

run-tests Trigger PR validation pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants