Fix/1924

[conradmugabe]

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

• Closes https://github.com/iblai/iblai-platform/issues/1924
• Add RBAC permissions to the datasets tab 'add resource button'

[mamigot]

Claude finished @conradmugabe's task —— View job

---

E2E Coverage Bot Analysis ✅

Reviewing PR #253 for e2e test coverage requirements.

Todo List:

• Step 1: Dismiss stale REQUEST_CHANGES reviews from myself (no existing reviews found)
• Step 2: Identify what changed (check for new page.tsx files)
• Step 3: Run mechanical coverage check
• Step 4: Read current coverage state
• Step 5: Apply decision logic and post GitHub review

Analysis:

Changed files:

• components/modals/edit-mentor-modal/tabs/datasets-tab/index.test.tsx (test file)
• components/modals/edit-mentor-modal/tabs/datasets-tab/index.tsx (component file)
• pnpm-lock.yaml (dependency lock file)

Key Finding: No new app/**/page.tsx files were added - this PR only modifies existing components within a modal tab.

Coverage Check Result: ❌ FAILED

• Checkpoints decreased from 442 to 434 (−8)
• Journey count decreased from 51 to 50

Decision: CASE D - Mechanical check FAILED (checkpoint regression)

Action Taken: Posted REQUEST_CHANGES review explaining that checkpoint count regression is not allowed and requesting the removed checkpoints be restored or justified.

---

[mamigot]

❌ E2E Checkpoint Regression

The checkpoint count in `e2e/coverage.json` decreased compared to `origin/main`.
Removing or reducing test coverage is not allowed.

Coverage check output:
```
Coverage REGRESSION: checkpoints decreased from 442 to 434 (−8)
You must not remove checkpoints without replacing them. Add tests for removed functionality or restore the checkpoints.
Journey REGRESSION: journey count decreased from 51 to 50
Journey coverage: 24/24 files mapped (100%)
Coverage check failed: new routes need E2E journey coverage.
```

To fix: restore the removed checkpoints or justify the removal by updating COVERAGE.md with a note.