Timmy_Foundation/hermes-agent
15
0
Fork 0
Code Issues 282 Pull Requests 87 Actions 1 Packages Projects Releases Wiki Activity

feat: add morning review packet generator (#966) #1025

Open
Rockachopa wants to merge 186 commits from fix/966 into main
pull from: fix/966
merge into: Timmy_Foundation:main
Conversation 1 Commits 186 Files Changed 152 +24712 -117
Rockachopa commented 2026-04-22 15:00:31 +00:00
Owner
Copy Link

Refs #966
Refs #949

This lands one concrete Phase 2 slice for the morning review packet epic instead of falsely claiming the whole phase is complete.

What landed

  • scripts/morning_review_packet.py — fetches a Gitea epic plus child QA issues, parses the structured sections in each child issue body, and renders a single review-ready markdown packet
  • docs/review_packets/hermes-harness-2026-04-21.md — generated packet artifact for the Hermes harness features landed 2026-04-21
  • tests/test_morning_review_packet.py — focused regression coverage for child-issue discovery, issue-range parsing, structured section extraction, and markdown rendering

Why this fits #966

Issue #966 is a broad Phase 2 tracker with no narrow acceptance contract in the issue body. This PR advances the phase by landing a reusable core logic primitive plus the concrete packet artifact for epic #949. The phase remains open because the actual QA execution across child issues #950-#962 is still outstanding.

Verification

  • python3 -m pytest -q tests/test_morning_review_packet.py
  • python3 -m py_compile scripts/morning_review_packet.py tests/test_morning_review_packet.py
  • python3 scripts/morning_review_packet.py --epic-number 949 --output docs/review_packets/hermes-harness-2026-04-21.md
Refs #966 Refs #949 This lands one concrete Phase 2 slice for the morning review packet epic instead of falsely claiming the whole phase is complete. ## What landed - `scripts/morning_review_packet.py` — fetches a Gitea epic plus child QA issues, parses the structured sections in each child issue body, and renders a single review-ready markdown packet - `docs/review_packets/hermes-harness-2026-04-21.md` — generated packet artifact for the Hermes harness features landed 2026-04-21 - `tests/test_morning_review_packet.py` — focused regression coverage for child-issue discovery, issue-range parsing, structured section extraction, and markdown rendering ## Why this fits #966 Issue #966 is a broad Phase 2 tracker with no narrow acceptance contract in the issue body. This PR advances the phase by landing a reusable core logic primitive plus the concrete packet artifact for epic #949. The phase remains open because the actual QA execution across child issues #950-#962 is still outstanding. ## Verification - `python3 -m pytest -q tests/test_morning_review_packet.py` - `python3 -m py_compile scripts/morning_review_packet.py tests/test_morning_review_packet.py` - `python3 scripts/morning_review_packet.py --epic-number 949 --output docs/review_packets/hermes-harness-2026-04-21.md`
Rockachopa added 1 commit 2026-04-22 15:00:32 +00:00
feat: add morning review packet generator (#966)
All checks were successful
Lint / lint (pull_request) Successful in 10s
Details
1bb48c91e7 
Implements one concrete Phase 2 slice for the morning review packet epic:
- add a script that fetches an epic and child QA issues from Gitea
- parse structured QA issue sections into a reusable packet model
- render a review-ready markdown packet
- add a generated 2026-04-21 Hermes harness review packet artifact
- cover parsing and rendering with targeted tests

Refs #966
Rockachopa reviewed 2026-04-22 15:12:10 +00:00
Rockachopa left a comment
Author
Owner
Copy Link

Review: Approved — useful scaffolding tool.

This PR adds a morning review packet generator and companion status reporter that pull from the Gitea API to build structured QA checklists.

Observations:

  1. The generated markdown packet is a static artifact — fine for archival.
  2. Token is read from ~/.config/gitea/token, consistent with other repo scripts.
  3. api_get has a nice fallback between token and Basic auth headers.
  4. SSL context uses ssl.create_default_context() which is correct.
  5. No sensitive data is hardcoded.

The doc-only nature of the output makes this low risk.

**Review: Approved — useful scaffolding tool.** This PR adds a morning review packet generator and companion status reporter that pull from the Gitea API to build structured QA checklists. **Observations:** 1. The generated markdown packet is a static artifact — fine for archival. 2. Token is read from `~/.config/gitea/token`, consistent with other repo scripts. 3. `api_get` has a nice fallback between `token` and `Basic` auth headers. 4. SSL context uses `ssl.create_default_context()` which is correct. 5. No sensitive data is hardcoded. The doc-only nature of the output makes this low risk.
All checks were successful
Lint / lint (pull_request) Successful in 10s
Details
This pull request has changes conflicting with the target branch.
  • cli.py
  • gateway/platforms/api_server.py
  • hermes_cli/models.py
  • hermes_cli/web_server.py
  • run_agent.py
  • tests/hermes_cli/test_web_server.py
  • tests/test_batch_executor.py
  • tools/batch_executor.py
  • tools/browser_tool.py
  • tools/file_tools.py
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin fix/966:fix/966
git checkout fix/966
Sign in to join this conversation.
Reviewers
No Reviewers
Rockachopa
Labels
Clear labels
CI
Continuous integration, runners, workflow issues
 QA
Quality assurance, testing, production audit
 accessibility
Auto-created for game epic
 assigned-claw-code
Queued for Code Claw (qwen/openrouter)
 assigned-kimi
Task assigned to KimiClaw for processing
 audit-finding
batch-pipeline
batch-pipeline label
 blocked
Blocked by external dependency or merge conflict
 bug
claw-code-done
Code Claw completed this task
 claw-code-in-progress
Code Claw is actively working
 critical
docs
Auto-created for game epic
 duplicate
Duplicate of another issue
 epic
Epic - large feature with multiple sub-tasks
 epic:memory
Unified Memory Architecture epic
 game
Auto-created for game epic
 gaming
Gaming agent capabilities
 implementation
Auto-created for game epic
 infra
Infrastructure, VPS, DNS, deployment
 infrastructure
Auto-created for game epic
 juice
Auto-created for game epic
 kimi-done
KimiClaw has completed this task
 kimi-in-progress
KimiClaw is actively working on this
 mcp
MCP (Model Context Protocol) tools & servers
 mechanics
Auto-created for game epic
 mobile
Auto-created for game epic
 morrowind
Morrowind Agent gameplay & MCP integration
 needs-review
PR or issue requires reviewer sign-off before merge/close
 ops
p0-critical
p1-important
performance
Auto-created for game epic
 phase:1-activate
Phase 1: Activate existing systems
 phase:2-operational
Phase 2: Operational memory bridge
 phase:3-episodic
Phase 3: Proactive episodic recall
 phase:4-hygiene
Phase 4: Memory hygiene and quality
 phase:5-integration
Phase 5: Integration and testing
 poka-yoke
polish
Auto-created for game epic
 priority-high
Auto-created for game epic
 priority-low
Auto-created for game epic
 priority-medium
Auto-created for game epic
 research
security
Security hardening, vulnerability fixes
 shared-infra
Shared infrastructure across projects (Playground + LPM)
 sovereign
Auto-created for game epic
 stale
No activity, pending triage or closure
 throughput-10x
throughput-10x label
 token-masterplan
token-masterplan label
 tool-evaluation
ux
Auto-created for game epic
 velocity-engine
Auto-generated by velocity engine
 visuals
Auto-created for game epic
 wont-fix
Closed as intentionally not fixing — explicit descope
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Rockachopa Timmy allegro antigravity bezalel claude claw-code codex-agent ezra gemini google grok hermes kimi manus perplexity sonnet
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Timmy_Foundation/hermes-agent#1025
Delete Branch "fix/966"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?