Skip to content

feat: migrate to pnpm, integrate CodeMirror 6, self-hosted fonts, URL sharing#1

Merged
cmurestudillos merged 1 commit into
masterfrom
develop
Jun 29, 2026
Merged

feat: migrate to pnpm, integrate CodeMirror 6, self-hosted fonts, URL sharing#1
cmurestudillos merged 1 commit into
masterfrom
develop

Conversation

@cmurestudillos

Copy link
Copy Markdown
Owner
  • Replace npm with pnpm 11.9.0 (packageManager field, pnpm-lock.yaml)
    • Update Husky hooks to use pnpm exec / pnpm run
    • Delete .eslintrc.json (Angular legacy, never linted React code)
    • Rewrite eslint.config.js with ESLint 9 flat config for React + TypeScript
    • Remove eslint-plugin-jsdoc and eslint-plugin-prettier from devDependencies
    • Integrate CodeMirror 6 (HTML/CSS/JS syntax highlighting, dark/light via Compartment)
    • Add manualChunks in vite.config.ts to split CodeMirror into separate chunk
    • Replace Google Fonts with @fontsource/raleway + @fontsource/fira-code (latin only)
    • Add URL sharing via base64 hash (TextEncoder/TextDecoder, replaceState)
    • Add share button with "¡Copiado!" feedback
    • Fix ConsolePanel: Tailwind dynamic hover classes, JSON object display, run separators
    • Fix iframe: remove deprecated frameBorder attribute
    • Fix loadTemplate/loadProject/resetCode: proper TypeScript types
    • Fix selector reset to "En Blanco" on load/reset
    • Update README with current stack, features, and install instructions
    • Add pr-review.yml workflow
    • Add pnpm-workspace.yaml

@cmurestudillos cmurestudillos self-assigned this Jun 29, 2026
@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown

🧪 Code Review Checklist

Antes de hacer merge, revisa:

  • He revisado mi propio código
  • No hay errores evidentes
  • He probado los cambios
  • El código sigue buenas prácticas
  • No introduzco bugs ni deuda técnica

💡 Recuerda: un buen PR = menos bugs en producción 🚀

@cmurestudillos cmurestudillos left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Revisado

@cmurestudillos
cmurestudillos merged commit 7458314 into master Jun 29, 2026
1 check 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