update import paths for workflow requests and remove index files

[asra-ameer]

Purpose

Remove index files in @wso2is/admin.workflow-requests.v1 package

Related Issues

wso2/product-is#25924

Related PRs

• N/A

Checklist

• e2e cypress tests locally verified. (for internal contributers)
• Manual test round performed and verified.
• UX/UI review done on the final implementation.
• Documentation provided. (Add links if there are any)
• Relevant backend changes deployed and verified
• Unit tests provided. (Add links if there are any)
• Integration tests provided. (Add links if there are any)

Security checks

• Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines
• Ran FindSecurityBugs plugin and verified report
• Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets

Developer Checklist (Mandatory)

• Complete the Developer Checklist in the related product-is issue to track any behavioral change or migration impact.

Summary by CodeRabbit

• Refactor
  • Optimized internal module structure and import paths for workflow requests functionality to improve code organization and maintainability.

[coderabbitai]

Walkthrough

The pull request refactors the workflow-requests feature module structure by removing intermediate re-export layers (deleting the pages aggregator file and removing public-api re-exports) while updating route imports to use more specific paths and supporting both default and named exports for compatibility.

Changes

Cohort / File(s)	Summary
Route configuration apps/console/src/configs/routes.tsx	Updated dynamic imports to reference specific page paths (workflow-requests/pages/workflow-requests and workflow-requests/pages/workflow-request-details); modified export resolution to support both default exports and named exports via fallback pattern (module?.default ?? module.WorkflowRequestsPage).
Module structure cleanup features/admin.workflow-requests.v1/index.ts	Removed Apache 2.0 license header and the re-export statement (export * from "./public-api").
Re-export aggregator removal features/admin.workflow-requests.v1/pages/index.ts	File deleted; removes public re-exports for WorkflowRequestsPage and WorkflowRequestDetailsPage default aliases.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

• Verify that the named export fallback pattern in routes.tsx correctly resolves exports from the targeted modules
• Confirm no external consumers depend on the removed re-export aggregators from features/admin.workflow-requests.v1/pages/index.ts and features/admin.workflow-requests.v1/index.ts
• Validate that lazy loading behavior remains intact with the updated import paths and dual export resolution

Poem

🐰 Hops through modules, pruning away the old,
Paths grow clearer, exports bold,
Re-exports trimmed, but function stays true,
A lighter structure, polished anew! ✨

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the main changes: updating import paths and removing index files in the workflow-requests module.
Description check	✅ Passed	The description includes the required Purpose section with clear intent, Related Issues link, and relevant checklist items completed appropriately.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 89a29e1 and d490f2f.

📒 Files selected for processing (3)

• apps/console/src/configs/routes.tsx (2 hunks)
• features/admin.workflow-requests.v1/index.ts (0 hunks)
• features/admin.workflow-requests.v1/pages/index.ts (0 hunks)

💤 Files with no reviewable changes (2)

• features/admin.workflow-requests.v1/index.ts
• features/admin.workflow-requests.v1/pages/index.ts

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: ʦ Typecheck (STATIC ANALYSIS) (lts/*, 8.7.4)

🔇 Additional comments (2)

apps/console/src/configs/routes.tsx (2)

1344-1346: ---

Code changes are correct; optional type safety improvement recommended.

The verification confirms that the target module exports WorkflowRequestsPage as a default export. The import pattern correctly handles this with module?.default, so the code is functionally sound. The fallback to module.WorkflowRequestsPage is unused but harmless.

Consider replacing the any type annotation with a more specific type to improve type safety:

import("@wso2is/admin.workflow-requests.v1/pages/workflow-requests").then((module: { default?: any; WorkflowRequestsPage?: any }) => ({
    default: module?.default ?? module.WorkflowRequestsPage
}))

Or if the module structure is fully known (a default export only):

import("@wso2is/admin.workflow-requests.v1/pages/workflow-requests").then((module: { default: any }) => ({
    default: module.default
}))

---

1363-1365: Code is correct—no changes needed.

The target module exports WorkflowRequestDetailsPage as a default export only. The implementation correctly retrieves it via module?.default, and the fallback pattern to the named export is harmless and defensive.

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 55.88%. Comparing base (75ecadc) to head (d490f2f).
⚠️ Report is 82 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #9424   +/-   ##
=======================================
  Coverage   55.88%   55.88%           
=======================================
  Files          42       42           
  Lines        1020     1020           
  Branches      254      254           
=======================================
  Hits          570      570           
+ Misses        439      416   -23     
- Partials       11       34   +23     

Flag	Coverage Δ
@wso2is/core	55.88% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 8 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[asra-ameer]

Hi @pavinduLakshan , just following up on the PR I sent last week. Whenever you get a moment, could you please review it and let me know if any changes are needed?