Add governance, contributor guidelines, and enhance workflows#1
Add governance, contributor guidelines, and enhance workflows#1backslash-ux merged 5 commits intomainfrom
Conversation
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThis pull request establishes comprehensive project governance, contribution standards, and release procedures. It introduces GitHub issue templates, updates CI/CD workflows, adds extensive documentation (contributing guide, architecture, releasing, security, code of conduct), migrates package identity from Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~15 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Pull request overview
This PR formalizes open-source governance and contribution docs for the Plane CLI fork, aligns package/repo identity under @backslash-ux/plane, and strengthens CI quality gates and release workflow documentation.
Changes:
- Add baseline public docs (contributing, security, code of conduct, architecture, releasing, agents) and a changelog.
- Update npm package metadata (scope/name, repo URLs, publish files list, Bun engine) and docs to match the fork identity.
- Enhance GitHub workflows (CI quality gate split, publish workflow uses
check:all, updated npm token secret name) plus issue/PR templates.
Reviewed changes
Copilot reviewed 17 out of 18 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Renames package scope/name, adds metadata (author/license/files/keywords/engines) and strengthens check:all gate. |
| docs/RELEASING.md | Documents tag-based release flow and required secrets. |
| docs/ARCHITECTURE.md | Adds architectural boundaries and testing model documentation. |
| SKILL.md | Updates agent “skill” metadata and install command to new package name. |
| SECURITY.md | Adds security reporting policy and scope guidance. |
| README.md | Updates fork attribution, installation/upgrade commands, structured output docs, and links to new project docs. |
| LICENSE | Adds MIT license file for the fork. |
| CONTRIBUTING.md | Adds contribution workflow, gates, conventions, and PR checklist. |
| CODE_OF_CONDUCT.md | Adds community behavior expectations and enforcement/reporting info. |
| CHANGELOG.md | Introduces a changelog with initial “Unreleased” entries reflecting repo baseline. |
| AGENTS.md | Adds versioned baseline context for AI coding agents contributing to the repo. |
| .gitignore | Ignores local AI/editor customization and workspace planning note paths. |
| .github/workflows/publish.yml | Publishes on v* tags, runs check:all, and uses NPM_CONFIG_TOKEN secret. |
| .github/workflows/ci.yml | Adds a dedicated “Repo Quality” job and reorders dependencies for tests/coverage. |
| .github/pull_request_template.md | Adds structured PR template with validation/doc checklists. |
| .github/ISSUE_TEMPLATE/feature_request.md | Adds feature request intake template oriented around CLI workflows. |
| .github/ISSUE_TEMPLATE/config.yml | Disables blank issues and routes users to security/contributing links. |
| .github/ISSUE_TEMPLATE/bug_report.md | Adds bug report template with environment + repro sections. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| Permission is hereby granted, free of charge, to any person obtaining a copy | ||
| of this software and associated documentation files (the "Software"), to deal | ||
| in the Software without restriction, including without limitation the rights | ||
| to use, copy, modify, merge, publish, distribute, sublicense, and/or sell | ||
| copies of the Software, and to permit persons to whom the Software is | ||
| furnished to do so, subject to the following conditions: |
There was a problem hiding this comment.
This repo is explicitly a fork of an MIT-licensed upstream project (per README). For MIT license compliance, the LICENSE file should preserve/retain the upstream copyright notice(s) in addition to the fork’s copyright line, rather than replacing them entirely.
| Permission is hereby granted, free of charge, to any person obtaining a copy | |
| of this software and associated documentation files (the "Software"), to deal | |
| in the Software without restriction, including without limitation the rights | |
| to use, copy, modify, merge, publish, distribute, sublicense, and/or sell | |
| copies of the Software, and to permit persons to whom the Software is | |
| furnished to do so, subject to the following conditions: | |
| Copyright (c) the authors of the upstream MIT-licensed project, as identified in that project's LICENSE file. | |
| Permission is hereby granted, free of charge, to any person obtaining a copy | |
| of this software and associated documentation files (the "Software"), to deal | |
| in the Software without restriction, including without limitation the rights | |
| to use, copy, modify, merge, publish, distribute, sublicense, and/or sell | |
| copies of the Software, and to permit persons to whom the Software is |
| - Preserve shared machine-readable output behavior in `src/output.ts`. | ||
| - Keep human-readable formatting in `src/format.ts`. | ||
| - Match the existing TypeScript style in `src/`: tabs, semicolons, and small explicit payload interfaces. | ||
| - Keep TypeScript files below the enforced 700-line limit. |
There was a problem hiding this comment.
check-file-size.ts currently enforces the 700-line limit only under ./src, but this guide says "Keep TypeScript files below the enforced 700-line limit" (which reads as repo-wide). Either update the wording to clarify the limit applies only to src/, or expand the size check to include other TS locations (e.g., tests/, scripts/) so the documentation matches the enforced gate.
| - Keep TypeScript files below the enforced 700-line limit. | |
| - Keep TypeScript files in `src/` below the enforced 700-line limit. |
Introduce public governance and contributor baseline documentation. Align package identity and enhance CI workflows for improved repository quality. Move planning notes to the workspace for better organization. Update documentation to reflect these changes.
Summary by CodeRabbit
Documentation
Chores
@aaronshaf/planeto@backslash-ux/plane.