Skip to content

Stabilize and modernize portfolio baseline - #21

Draft
JMG3000 wants to merge 1 commit into
mainfrom
codex/portfolio-modernization
Draft

Stabilize and modernize portfolio baseline#21
JMG3000 wants to merge 1 commit into
mainfrom
codex/portfolio-modernization

Conversation

@JMG3000

@JMG3000 JMG3000 commented Aug 7, 2026

Copy link
Copy Markdown
Owner

Tracking

Summary

  • Replace the mixed legacy/starter portfolio state with a responsive Next.js portfolio baseline while preserving project history.
  • Make GitHub Pages static export deterministic with focused tests, cross-platform validation, and least-privilege deployment controls.
  • Remediate the dependency boundary with Next.js 16.2.12, js-yaml 5.2.3, eslint-config-next 16.2.12, PostCSS 8.5.23, and a regenerated reproducible lockfile.

Root cause

The repository combined retained Jekyll artifacts, generated Next.js output, placeholder routes/content, and a request-time API route that cannot run on static GitHub Pages. A newly disclosed dependency advisory also made the earlier publication gate fail. A global brace-expansion override was rejected because it breaks minimatch 3's CommonJS API contract; the resolved dependency graph now uses compatible patched versions without an adapter.

Validation

  • npm ci — clean install from the committed lockfile
  • npm audit --omit=dev --audit-level=high — 0 vulnerabilities
  • npm audit --audit-level=high — 0 vulnerabilities
  • npm run check — lint, typecheck, 5 test files / 11 tests, static production build
  • Static routes: /, /404, /portfolio

@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 15b90300-6440-410b-b27f-2ab8891d58c5

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

Comment @coderabbitai help to get the list of available commands.

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