Skip to content

refactor: remove Turborepo monorepo structure, convert to pure OSS template#29

Merged
p-iknow merged 1 commit intomainfrom
i27-feat/add-agents-md-scope-docs
Jan 24, 2026
Merged

refactor: remove Turborepo monorepo structure, convert to pure OSS template#29
p-iknow merged 1 commit intomainfrom
i27-feat/add-agents-md-scope-docs

Conversation

@p-iknow
Copy link
Contributor

@p-iknow p-iknow commented Jan 24, 2026

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

  • Remove turbo.json and pnpm-workspace.yaml (monorepo config)
  • Delete apps/ directory (web, docs Next.js applications)
  • Delete packages/ directory (ui, eslint-config, typescript-config shared packages)
  • Simplify package.json to template-only with minimal dependencies (prettier for formatting)
  • Regenerate pnpm-lock.yaml without Turborepo dependencies
  • Add scoped AGENTS.md files:
    • Root AGENTS.md - Project knowledge base
    • .claude/AGENTS.md - Claude Code integration documentation
    • .github/AGENTS.md - GitHub configuration documentation

Test

N/A - No test files affected (template repository)

Human Check

🤖 Generated with Claude Code

  • I reviewed it

…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
@github-actions github-actions bot added the refactor Code refactoring label Jan 24, 2026
@p-iknow p-iknow marked this pull request as ready for review January 24, 2026 09:28
Copilot AI review requested due to automatic review settings January 24, 2026 09:28
@p-iknow p-iknow merged commit f72178d into main Jan 24, 2026
8 checks passed
@p-iknow p-iknow deleted the i27-feat/add-agents-md-scope-docs branch January 24, 2026 09:28
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

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
Copy link

Copilot AI Jan 24, 2026

Choose a reason for hiding this comment

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

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.

Copilot uses AI. Check for mistakes.
Comment on lines +31 to +38
| 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) |
Copy link

Copilot AI Jan 24, 2026

Choose a reason for hiding this comment

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

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.

Suggested change
| 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) |

Copilot uses AI. Check for mistakes.
Comment on lines +36 to +37
bun .claude/skills/meta-skill-creator/scripts/init-skill.ts <name> --path <dir>
bun .claude/skills/meta-skill-creator/scripts/validate-skill.ts <folder>
Copy link

Copilot AI Jan 24, 2026

Choose a reason for hiding this comment

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

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.

Copilot uses AI. Check for mistakes.
Comment on lines +45 to +46
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>
Copy link

Copilot AI Jan 24, 2026

Choose a reason for hiding this comment

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

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.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Code refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: add AGENTS.md documentation for each scope level

1 participant

Comments