chore: remove dev flag from identity

[MayankBansal12]

Date: 01-01-2026

Developer Name: @MayankBansal12

---

Issue Ticket Number:-

• Remove Feature Flag from /identity page #1121

Description:

• Remove feature flag from identity page
• remove test for identity route for dev flag

Is Under Feature Flag

• Yes
• No

Database changes

• Yes
• No

Breaking changes (If your feature is breaking/missing something please mention pending tickets)

• Yes
• No

Is Development Tested?

• Yes
• No

Tested in staging?

• Yes
• No

Add relevant Screenshot below ( e.g test coverage etc. )

screencast

demo-identity.mp4

[cloudflare-workers-and-pages]

Deploying www-rds with    Cloudflare Pages

Latest commit:	9284cc7
Status:	🚫  Build failed.

View logs

[coderabbitai]

Walkthrough

The IdentityRoute's beforeModel lifecycle hook, which conditionally redirected to /page-not-found based on a dev query parameter, is removed. The corresponding unit tests validating this redirect behavior are also removed, allowing the model() flow to execute unconditionally.

Changes

Cohort / File(s)	Summary
Route lifecycle hook removal app/routes/identity.js	Removed beforeModel(transition) lifecycle hook that redirected to /page-not-found unless transition.to.queryParams.dev === 'true'. Model flow now runs unconditionally (subject to fastboot and subsequent checks).
Test cleanup tests/unit/routes/identity-test.js	Removed two unit tests that validated beforeModel behavior: one asserting redirect when dev parameter is not true, another asserting no redirect when dev is true.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 No more gates to block the way,
The dev parameter fades away.
Model flows without a care,
Freedom blooms in routes laid bare!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'chore: remove dev flag from identity' accurately and concisely summarizes the main change: removing the dev flag feature from the identity route.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check	✅ Passed	The pull request description clearly describes removing the dev feature flag from the identity page and associated tests, which directly aligns with the code changes in the changeset.

✨ Finishing touches

• 📝 Generate docstrings

---

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.}