Conversation
Co-authored-by: LLxD <48529975+LLxD@users.noreply.github.com>
Co-authored-by: LLxD <48529975+LLxD@users.noreply.github.com>
Co-authored-by: LLxD <48529975+LLxD@users.noreply.github.com>
Co-authored-by: LLxD <48529975+LLxD@users.noreply.github.com>
Co-authored-by: LLxD <48529975+LLxD@users.noreply.github.com>
Co-authored-by: LLxD <48529975+LLxD@users.noreply.github.com>
Co-authored-by: LLxD <48529975+LLxD@users.noreply.github.com>
|
@copilot make sure we're well covered for multiple operational systems with this, users might add multiple skills and front-end guidances as they see fit, based on their AI of choice and it's configuration (e.g. cursorrules, copilot-instructions, AGENTS.md, etc) |
|
@copilot also, properly structure this project so that it's always installed and used like rorg, instead of @llxd/rorg or anything like that. structure proper CI versioning and all necessary for quality of life in this library development |
Co-authored-by: LLxD <48529975+LLxD@users.noreply.github.com>
Co-authored-by: LLxD <48529975+LLxD@users.noreply.github.com>
... Done in commit Cross-platform OS coverage:
AI Directions (
27 new tests added (107 total, all green). 0 security vulnerabilities. |
Co-authored-by: LLxD <48529975+LLxD@users.noreply.github.com>
Done in commit Package always installs as
CI — Release —
Requires an Dev QoL:
|
This PR addresses missing test coverage, weak input validation, inadequate error handling, cross-platform compatibility, adds a new AI directions generator, and establishes proper package structure with CI/CD for library development.
Test Infrastructure
npm test,npm run test:watch,npm run test:coverageUtility Modules
utils/validation.js< > : " | ? * / \), path traversal attempts, enforces 255 char limitutils/fileSystem.jsError Handling
/.+/) with comprehensive sanitizationCross-Platform OS Support
utils/validation.js: normalized paths always use forward slashes regardless of OS (.replace(/\\/g, '/'))C:/,C:\), backslash-separated paths, and guarantee forward-slash output on all platforms[/\\]) instead of hardcoded/AI Directions Generator (
npx rorg ai)A new modular generator that creates AI assistant configuration files pre-populated with the project's architectural pattern, base path, naming conventions, and React/TypeScript guidelines.
Supported tools:
.cursorrules.github/copilot-instructions.md.windsurfrulesAGENTS.md.continuerulesRun
npx rorg aiafter initialization to pick one or more tools via a checkbox prompt. Each file is automatically tailored to your project's settings.Package Structure & npm Publishing
package.jsonnow includes afilesfield (publishes onlybin/,generators/,plop-templates/,utils/,plopfile.js,README.md,LICENSE)publishConfigset to{ "access": "public", "registry": "https://registry.npmjs.org/" }— ensures the package is always published as the unscopedrorg, never as@llxd/rorgengines: { "node": ">=18.0.0" }makes Node.js requirements explicitprepublishOnly: "npm test"blocks publishing if tests failhomepageandbugsfields; structuredrepositoryobjectCI/CD Workflows
.github/workflows/ci.yml— runs on every push tomain/masterand every PR:.github/workflows/release.yml— two release modes:v*tag → tests run → publish to npm → GitHub Release created with auto-generated notesworkflow_dispatchwithpatch/minor/majoror exact version → bumpspackage.json, creates and pushes tag, publishes to npm, creates GitHub ReleaseRequires an
NPM_TOKENsecret set in repository settings.Developer Quality of Life
CONTRIBUTING.mdfully rewritten: npm setup (replaces pnpm), project structure map, versioning guide using the new workflows, instructions for adding generators and AI toolsCHANGELOG.mdadded (Keep a Changelog format)README.mdnow shows CI, npm version, and MIT badges; documents bothnpx rorgandnpm install -g rorgBug Fixes
pages-components.js: Now respectsseparateCssconfig flag (was always creating CSS files)path.join()for cross-platform compatibilitySecurity
Example
Before:
After:
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.