[INTF26] Rebuilds review UI revamp on staging

[OpsEclipse]

Notion ticket link

Revamp Review Basic Info Page Layout

Implementation description

• Rebuilds the review UI revamp on top of the latest staging branch so the PR keeps the intended design changes without carrying the old conflict-heavy history forward.
• Restores the revamped review experience across the basic info page, scored review stages, final scoring screen, and submit-success screen using review-specific layouts and updated answer/rubric presentation.
• Adds safer review parsing and score progression guardrails so reviewers cannot skip ahead through required scoring stages from either the header stepper or footer controls.

Steps to test

1. Run ./node_modules/.bin/eslint on the touched review files and confirm it passes.
2. Run yarn lint:tsc and note the current failures in components/admin/ReviewTable.tsx, components/admin/ReviewTableColumn.tsx, components/interview/shared/InterviewProgressContext.tsx, and components/interview/shared/types.ts, which are outside this PR diff.
3. Run yarn dev, open a valid review link, and verify the revamped UI across the basic info page, PFSG, team player, drive to learn, skill, final scoring, and submit-success states.
4. Try advancing without entering a score and confirm the flow blocks progress from both the header and footer navigation until the required stage is scored.
5. Open /review without a valid reviewId and confirm the invalid-link state is shown.

What should reviewers focus on?

• Whether the rebuilt review-specific layouts preserve the intended INTF26 revamp while fitting cleanly on top of the newer staging layout work.
• The new shared score/progress helpers in the review flow, especially header navigation limits and final submission gating.
• The refreshed final scoring and submit-success UI.

Checklist

• My PR name is descriptive and in imperative tense
• My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
• I have run the appropriate linter(s)
• I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

[mxc-maggiechen]

I think we need the number cast here, when retrieving the ID from local host storage, the id gets typed as string during run time, even though at compile time it is typed as number