agent: Tasks - coding-agent - replace progress.txt notes with Git commits. Actu

[sweetmantech]

Automated PR from coding agent.

Prompt: Tasks - coding-agent - replace progress.txt notes with Git commits. Actual: prefix and suffix in prompt reference progress.txt. Required: replace unreliable progress.txt file with note to read latest commits to understand latest changes and note to commit any changes so the next dev knows what was worked on.

Summary by CodeRabbit

• Refactor
  • Updated progress tracking mechanism to leverage git operations instead of file-based approach.
  • Enhanced commit workflow to generate descriptive commit messages automatically.

[coderabbitai]

📝 Walkthrough

Walkthrough

Two exported prompt prefix constants in src/tasks/promptPrefixes.ts have their values updated. The PROGRESS_PROMPT_PREFIX now instructs git log execution in submodules instead of reading markdown files, while PROGRESS_PROMPT_SUFFIX now directs committing changes with descriptive messages instead of updating markdown documentation.

Changes

Cohort / File(s)	Summary
Prompt Prefix Constants src/tasks/promptPrefixes.ts	Updated PROGRESS_PROMPT_PREFIX to run git log in submodules; updated PROGRESS_PROMPT_SUFFIX to commit changes with descriptive messages instead of updating markdown files.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• agent: @U0AJM7X8FBR mono/tasks codebase - In updatePRTask.ts, the feedback mess #97: Directly related as it introduced/extracted the PROGRESS_PROMPT_PREFIX constant that is being updated in this PR.
• fix: push mono repo root progress files directly to main #96: Related as it also modifies agent prompt text controlling submodule and commit handling behavior.

Poem

🐰 A prompt gets a gentle tune,
Git log now sings its moon-bright croon,
Submodules dance, commits take flight,
Markdown rests—the CLI shines bright! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title mentions the main change (replacing progress.txt notes with Git commits) but is incomplete/truncated at the end ('Actu') and does not fully convey the complete intent.	Complete the truncated title to clearly express the full change, for example: 'agent: Tasks - coding-agent - replace progress.txt notes with Git commits'

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch agent/tasks---coding-agent---replace-1773869575358

📝 Coding Plan

• Generate coding plan for human review comments

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

🧹 Nitpick comments (1)

src/tasks/promptPrefixes.ts (1)

5-5: Make the commit instruction conditional.

Line 5 currently implies commit is always required. If no edits are needed, that can create unnecessary commit-step failures/no-op behavior. Prefer conditional wording.

Proposed prompt text tweak

 export const PROGRESS_PROMPT_SUFFIX =
-  "Commit all your changes with a descriptive message so the next developer knows what was worked on.";
+  "If you made changes, commit them with a descriptive message so the next developer knows what was worked on; if no changes were needed, explicitly state that.";

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/tasks/promptPrefixes.ts` at line 5, The prompt string "Commit all your
changes with a descriptive message so the next developer knows what was worked
on." should be made conditional to avoid forcing a commit when no edits were
made; update the prompt in src/tasks/promptPrefixes.ts (the string literal
above) to something like "If you made changes, commit them with a descriptive
message so the next developer knows what was worked on; otherwise, no commit is
necessary." ensuring the wording clearly advises skipping the commit when there
are no edits.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@src/tasks/promptPrefixes.ts`:
- Line 5: The prompt string "Commit all your changes with a descriptive message
so the next developer knows what was worked on." should be made conditional to
avoid forcing a commit when no edits were made; update the prompt in
src/tasks/promptPrefixes.ts (the string literal above) to something like "If you
made changes, commit them with a descriptive message so the next developer knows
what was worked on; otherwise, no commit is necessary." ensuring the wording
clearly advises skipping the commit when there are no edits.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: bb1c8351-a30b-4fc4-986b-90ce785247de

📥 Commits

Reviewing files that changed from the base of the PR and between 1823748 and 6f17346.

📒 Files selected for processing (1)

• src/tasks/promptPrefixes.ts