diff --git a/AGENTS.md b/AGENTS.md index 0d53b4c..81f54c7 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -8,6 +8,34 @@ When working in this repository: - Do not revert unrelated user changes. - Prefer opening a pull request instead of pushing directly to `main`, unless the user explicitly asks to push to `main`. +## Issue-First Pull Request Workflow + +Use an issue before opening a pull request for: + +- bug reports and bug fixes; +- reliability or security fixes; +- MVP features or launch-checklist work; +- user-facing UX or terminal-output changes. + +Workflow: + +1. Read the PRD and relevant documentation. +2. Create a `codex/` branch and prepare the implementation locally. +3. Add or update tests, documentation, and personal development notes. +4. Run the relevant verification gates. +5. Before committing, pushing, or opening the PR, stop and ask the user to + create a GitHub issue. Provide a ready-to-use issue title and body. +6. Wait for the user to provide the issue number. +7. Commit and push the verified work, then open a PR that includes + `Closes #` in the PR body. +8. Give the user the PR link so they can review and merge it. The merge should + close the linked issue automatically. + +Do not create the GitHub issue on the user's behalf unless they explicitly ask. +Small documentation-only, repository-process, typo, dependency-maintenance, or +CI housekeeping changes may proceed without an issue unless the user requests +one. + ## Commits * Keep the user's Git identity as the primary author whenever possible. diff --git a/docs/for-me-personal/PROGRESS.md b/docs/for-me-personal/PROGRESS.md index dbd76a8..6d7e28d 100644 --- a/docs/for-me-personal/PROGRESS.md +++ b/docs/for-me-personal/PROGRESS.md @@ -30,6 +30,18 @@ Terakhir diperbarui: 2026-06-14 - Renderer panel dipisahkan agar layout lebar dan sempit dapat diuji langsung. - Automated test saat ini berjumlah 49 dan seluruhnya lulus. +### Issue-First PR Workflow + +- Bug, reliability/security fix, fitur MVP, dan perubahan UX sekarang memakai + issue GitHub sebelum PR dibuat. +- Implementasi dan verification boleh diselesaikan lokal terlebih dahulu. +- Sebelum commit, push, atau PR, agent berhenti dan memberi title/body issue + siap pakai kepada maintainer. +- Setelah nomor issue diberikan, PR harus memakai `Closes #N` agar issue + tertutup otomatis ketika merge. +- Perubahan dokumentasi/proses kecil, typo, dependency maintenance, dan CI + housekeeping tidak wajib memakai issue. + ## Update 2026-06-13 ### Packed CLI End-to-End