Danny Kim
ccfb08f4d5
Follow-up to #20, which was merged before this landed. bot-agents has no `package.json` `imports` aliases (`src/` is flat and everything imports relatively), so no code changes. This records the new rule in the docs: - `.agents/memory/imports-aliases-barrels.md`: aliases may target only a direct subfolder of `src/` (its `index.ts` or a `#x/*` pattern); nothing outside `src/`, not `src/` itself, not deeper folders; such code imports relatively. - `.agents/memory/tests-layout-vitest.md`: `tests/` reach the public entry through a relative import, not a `#plugin` alias. - `.agents/skills/migrate-deno-repo/SKILL.md` and `AGENTS.md`: same rule. `vp check`, `vp test`, and `damem doctor` pass. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Reviewed-on: #21 Reviewed-by: bot <noreply@capsulizers.com> Co-authored-by: Danny Kim <temeddix@gmail.com> Co-committed-by: Danny Kim <temeddix@gmail.com>
2.2 KiB
2.2 KiB
description
| description |
|---|
| Work on a branch, plain imperative commits without AI trailers, one PR with the Claude Code footer, review until approved, never merge yourself; squash only |
Branches and commits
- Never push to
main. Every change goes through a branch and a PR, even a one-line fix. - Commit messages are plain imperative sentences ("Port the import-check
rules to an ESLint plugin"). No conventional-commit prefixes, no AI
trailers (
Co-Authored-By: Claude, "Generated with ..."), no emoji. - When several agents share a branch, each works in its own
git worktree, runsgit pull --rebasebefore pushing, and retries a rejected push after rebasing. Remove the worktree and temp branch afterwards. vp check,vp test, and the build pass locally before every push.
Pull requests
-
One PR per repo for a migration, everything included (size-rule refactors too).
-
The body covers: summary, what changed, deviations and risks, verification done, and open questions. It ends with exactly this line:
🤖 Generated with [Claude Code](https://claude.com/claude-code) -
Then run the review cycle (skill
review-cycle): on Capsulizers Gitea repos that have.gitea/workflows/agents.yml, comment@botand address thebotaccount's review; elsewhere, a subagent reviews the diff and posts the review as a PR comment. Repeat until fully approved, then ask the user to review.
Merging
- Never merge on your own. The user merges, unless they explicitly allow it for one specific PR.
- GitHub repos are squash-merge only, so a PR lands as one commit titled after the PR.
- Also never, without the user: publish, archive a repo, or delete a branch the user may still use.
Hosts
- cunarist and temeddix: GitHub only, CI in
.github/workflows, use theghCLI. - Capsulizers: Gitea at git.capsulizers.com (Gitea 1.26), CI in
.gitea/workflows, PRs through the Gitea API. Push and open PRs as the user's own account (credentials in the user's environment; source them in a subshell and never print them);botis a separate review account. The org's codified rules live in thecommons/code-rulesREADME.