Update remind-sprint-update.sh hook text for boards #83

Merged
forgejo_admin merged 1 commit from 82-board-hook-text into main 2026-03-13 19:46:23 +00:00
Contributor

Summary

Updates the remind-sprint-update.sh hook text to use board terminology instead of sprint terminology, aligning with the renamed pal-e-docs MCP tools.

Changes

  • hooks/remind-sprint-update.sh -- replaced all "sprint" references with "board" equivalents:
    • Comment: "sprint item" -> "board item"
    • Comment: "sprint board" -> "board"
    • Reminder text: "Sprint board reminder" -> "Board reminder"
    • Reminder text: "current sprint" -> "current board"
    • Reminder text: "sprint item" -> "board item"
    • Tool reference: mcp__pal-e-docs__move_sprint_item -> mcp__pal-e-docs__move_board_item

Test Plan

  • Verify the hook still produces valid JSON output: bash hooks/remind-sprint-update.sh | jq .
  • Confirm no remaining "sprint" references in the reminder text

Review Checklist

  • Text-only change, no logic modifications
  • JSON output structure unchanged
  • Tool name matches renamed MCP tool (move_board_item)
## Summary Updates the `remind-sprint-update.sh` hook text to use board terminology instead of sprint terminology, aligning with the renamed pal-e-docs MCP tools. ## Changes - `hooks/remind-sprint-update.sh` -- replaced all "sprint" references with "board" equivalents: - Comment: "sprint item" -> "board item" - Comment: "sprint board" -> "board" - Reminder text: "Sprint board reminder" -> "Board reminder" - Reminder text: "current sprint" -> "current board" - Reminder text: "sprint item" -> "board item" - Tool reference: `mcp__pal-e-docs__move_sprint_item` -> `mcp__pal-e-docs__move_board_item` ## Test Plan - Verify the hook still produces valid JSON output: `bash hooks/remind-sprint-update.sh | jq .` - Confirm no remaining "sprint" references in the reminder text ## Review Checklist - [x] Text-only change, no logic modifications - [x] JSON output structure unchanged - [x] Tool name matches renamed MCP tool (`move_board_item`) ## Related - Closes #82
Aligns hook reminder text with the renamed pal-e-docs MCP tools
(move_sprint_item -> move_board_item) and updated board terminology.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Author
Contributor

PR #83 Review

BLOCKERS

None.

NITS

  1. Hook filename still says "sprint" -- remind-sprint-update.sh was not renamed to reflect the board terminology. Not blocking since the file works fine, but worth a follow-up issue if you want full consistency.
  2. /sprint-sync reference retained -- The additionalContext string still says "Use /sprint-sync or manually call...". The /sprint-sync skill exists at skills/sprint-sync/SKILL.md and has not been renamed, so this reference is currently correct. If/when the skill gets renamed, this hook will need another text pass.

SOP COMPLIANCE

  • Branch named after issue (82-board-hook-text references #82)
  • PR body follows template (Summary, Changes, Test Plan, Related)
  • Closes #82 present in Related section
  • No secrets committed
  • No scope creep (1 file, 3 lines changed, text-only)
  • Commit message is descriptive

VERDICT: APPROVED

Clean text-only change. All "sprint" references in the reminder output and comments have been replaced with "board" equivalents. The MCP tool name correctly updated to mcp__pal-e-docs__move_board_item. JSON structure is unchanged. The two nits (filename, /sprint-sync command name) are out of scope for this PR and can be addressed separately.

## PR #83 Review ### BLOCKERS None. ### NITS 1. **Hook filename still says "sprint"** -- `remind-sprint-update.sh` was not renamed to reflect the board terminology. Not blocking since the file works fine, but worth a follow-up issue if you want full consistency. 2. **`/sprint-sync` reference retained** -- The `additionalContext` string still says "Use /sprint-sync or manually call...". The `/sprint-sync` skill exists at `skills/sprint-sync/SKILL.md` and has not been renamed, so this reference is currently correct. If/when the skill gets renamed, this hook will need another text pass. ### SOP COMPLIANCE - [x] Branch named after issue (`82-board-hook-text` references #82) - [x] PR body follows template (Summary, Changes, Test Plan, Related) - [x] `Closes #82` present in Related section - [x] No secrets committed - [x] No scope creep (1 file, 3 lines changed, text-only) - [x] Commit message is descriptive ### VERDICT: APPROVED Clean text-only change. All "sprint" references in the reminder output and comments have been replaced with "board" equivalents. The MCP tool name correctly updated to `mcp__pal-e-docs__move_board_item`. JSON structure is unchanged. The two nits (filename, `/sprint-sync` command name) are out of scope for this PR and can be addressed separately.
forgejo_admin deleted branch 82-board-hook-text 2026-03-13 19:46:23 +00:00
Sign in to join this conversation.
No description provided.