Skip to content

Add Zyra Job Readiness command center - #35

Merged
sonoxo merged 6 commits into
mainfrom
feature/job-readiness
Aug 24, 2026
Merged

Add Zyra Job Readiness command center#35
sonoxo merged 6 commits into
mainfrom
feature/job-readiness

Conversation

@sonoxo

@sonoxo sonoxo commented Aug 24, 2026

Copy link
Copy Markdown
Owner

What this adds

  • First-class /job-readiness workspace for AI/software, cloud/DevSecOps, cybersecurity, and platform career tracks
  • Portfolio-proof links into existing Zyra modules
  • Hiring/readiness checklist and interview translation guidance
  • docs/JOB_READINESS.md engineering standard
  • Job Readiness CI gate requiring clean install, TypeScript validation, and production build
  • /career redirect for a simple entry point

Goal

Make Zyra easier to present as verifiable engineering work to recruiters and hiring managers without weakening its existing security/enterprise focus. The standard emphasizes reproducible evidence, clean builds, secure public portfolio material, and honest claims.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7e2e177485

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +27 to +28
label: "Production engineering",
score: 88,

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Derive readiness scores from verified evidence

Every installation, including a fresh tenant with no portfolio evidence or a currently failing build, receives the same hard-coded scores and an overall value of 86, even though the UI labels this as “Current readiness” and presents it as employer-facing evidence. These values should be calculated from verifiable repository/account state, or clearly presented as non-evaluative guidance rather than readiness measurements.

Useful? React with 👍 / 👎.

Comment thread client/src/App.tsx
Comment on lines +180 to +181
<Route path="/job-readiness">{() => <ProtectedRoute component={JobReadinessPage} />}</Route>
<Route path="/career">{() => <Redirect to="/job-readiness" />}</Route>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Add the workspace to application navigation

Authenticated users who navigate through the Layout sidebar have no way to discover this route: navGroups contains no Job Readiness entry, and a repository-wide search finds no other link to /job-readiness or /career. Registering the route alone therefore leaves the new first-class workspace accessible only to users who already know and manually enter its URL.

Useful? React with 👍 / 👎.

Comment on lines +225 to +226
{proofRoutes.map(({ label, href, icon: Icon, detail }) => (
<Link key={href} href={href}>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Add stable test IDs to the portfolio links

The six proof tiles are interactive navigation elements but lack unique data-testid attributes, so browser tests following the project's selector convention cannot reliably identify or exercise each destination; the final “Review proof” action has the same omission. Add a route- or label-derived test ID to each link/action.

Useful? React with 👍 / 👎.

@sonoxo
sonoxo merged commit 967c593 into main Aug 24, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant