Conversation
…mplate Requirement: Repository should be a lightweight GitHub template for OSS project setup, not a Turborepo monorepo boilerplate with sample apps. Implementation: - Remove turbo.json and pnpm-workspace.yaml - Delete apps/ directory (web, docs Next.js apps) - Delete packages/ directory (ui, eslint-config, typescript-config) - Simplify package.json to template-only (prettier for formatting) - Regenerate pnpm-lock.yaml without turbo dependencies - Add scoped AGENTS.md files for project, .claude/, and .github/ directories
There was a problem hiding this comment.
Pull request overview
This pull request converts the repository from a Turborepo monorepo boilerplate to a lightweight GitHub template focused on open-source project configurations. The changes address issue #27 by removing all monorepo infrastructure and adding comprehensive AGENTS.md documentation files.
Changes:
- Removed Turborepo monorepo structure (turbo.json, pnpm-workspace.yaml, apps/, packages/)
- Simplified package.json to template-only with minimal dependencies
- Added three scoped AGENTS.md files documenting project structure, GitHub workflows, and Claude Code integration
Reviewed changes
Copilot reviewed 41 out of 62 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| turbo.json | Removed Turborepo configuration file |
| pnpm-workspace.yaml | Removed pnpm workspace configuration |
| pnpm-lock.yaml | Regenerated with only prettier dependency (version 3.8.1) |
| packages/* | Removed all shared packages (ui, eslint-config, typescript-config) |
| apps/* | Removed all applications (web, docs Next.js apps) |
| package.json | Updated name to @p-stack/oss-kit, cleaned keywords, simplified scripts |
| AGENTS.md | Added root-level project knowledge base documentation |
| .github/AGENTS.md | Added GitHub configuration documentation |
| .claude/AGENTS.md | Added Claude Code integration documentation |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| typescript: | ||
| specifier: 5.9.2 | ||
| version: 5.9.2 | ||
| version: 3.8.1 |
There was a problem hiding this comment.
The prettier version specification has been changed from "^3.6.2" to "^3.6.2" in the package.json (line 18), but the lock file shows it resolved to version 3.8.1 (line 13 of pnpm-lock.yaml). This is a significant version jump (3.6.2 -> 3.8.1) which could potentially introduce formatting changes. While this is within the caret range (^3.6.2 allows for 3.x.x updates), it's worth noting that the actual installed version differs from what might be expected from the package.json specification.
| | Task | Location | Notes | | ||
| | ------------------- | ---------------------------------- | ----------------------- | | ||
| | Add issue template | `.github/ISSUE_TEMPLATE/` | Match commit type | | ||
| | Modify PR template | `.github/pull_request_template.md` | Single template | | ||
| | Add GitHub workflow | `.github/workflows/` | auto-assign, auto-label | | ||
| | Add Claude command | `.claude/commands/` | Markdown format | | ||
| | Add Claude skill | `.claude/skills/` | Use meta-skill-creator | | ||
| | Update docs | `docs/` | Bilingual (en/ko) | |
There was a problem hiding this comment.
In the table on line 31-38, the "Location" column uses absolute paths starting from the repository root (e.g., ".github/ISSUE_TEMPLATE/"), but the "Task" descriptions use relative language like "Add issue template". For consistency with how users would navigate, consider whether these paths should remain as-is or be described more consistently throughout the document.
| | Task | Location | Notes | | |
| | ------------------- | ---------------------------------- | ----------------------- | | |
| | Add issue template | `.github/ISSUE_TEMPLATE/` | Match commit type | | |
| | Modify PR template | `.github/pull_request_template.md` | Single template | | |
| | Add GitHub workflow | `.github/workflows/` | auto-assign, auto-label | | |
| | Add Claude command | `.claude/commands/` | Markdown format | | |
| | Add Claude skill | `.claude/skills/` | Use meta-skill-creator | | |
| | Update docs | `docs/` | Bilingual (en/ko) | | |
| | Task | Repository path (from root) | Notes | | |
| | ------------------- | ---------------------------------- | ----------------------- | | |
| | Add issue template | `./.github/ISSUE_TEMPLATE/` | Match commit type | | |
| | Modify PR template | `./.github/pull_request_template.md` | Single template | | |
| | Add GitHub workflow | `./.github/workflows/` | auto-assign, auto-label | | |
| | Add Claude command | `./.claude/commands/` | Markdown format | | |
| | Add Claude skill | `./.claude/skills/` | Use meta-skill-creator | | |
| | Update docs | `./docs/` | Bilingual (en/ko) | |
| bun .claude/skills/meta-skill-creator/scripts/init-skill.ts <name> --path <dir> | ||
| bun .claude/skills/meta-skill-creator/scripts/validate-skill.ts <folder> |
There was a problem hiding this comment.
The meta-skill-creator examples use "bun" as the command runner (lines 36-37), but the root package.json and pnpm-lock.yaml indicate this project uses pnpm as the package manager. There's no bun configuration or dependency in the project. Either these commands should use a different runner that's actually configured in the project, or bun should be added as a dependency if it's required for these scripts to work.
| bun .claude/skills/meta-agent-creator/scripts/init-agent.ts <name> --path <dir> --platform <platform> | ||
| bun .claude/skills/meta-agent-creator/scripts/validate-agent.ts <file> |
There was a problem hiding this comment.
Similar to the meta-skill-creator section, the meta-agent-creator examples also use "bun" as the command runner (lines 45-46), but this is inconsistent with the project's use of pnpm. This will cause confusion for users trying to run these commands.
Requirement
resolves #27
Convert repository from a Turborepo monorepo boilerplate to a lightweight GitHub template focused solely on OSS project configuration (issue templates, PR templates, workflows, Claude Code integration).
Implementation
turbo.jsonandpnpm-workspace.yaml(monorepo config)apps/directory (web, docs Next.js applications)packages/directory (ui, eslint-config, typescript-config shared packages)package.jsonto template-only with minimal dependencies (prettier for formatting)pnpm-lock.yamlwithout Turborepo dependenciesAGENTS.mdfiles:AGENTS.md- Project knowledge base.claude/AGENTS.md- Claude Code integration documentation.github/AGENTS.md- GitHub configuration documentationTest
N/A - No test files affected (template repository)
Human Check
🤖 Generated with Claude Code