coilyco-flight-deck/agentic-os
2
0
Fork 0
Code Issues 32 Pull requests 2 Projects Releases 230 Packages 1 Activity Actions 2

Configure warded git committer identity #508

Merged
coilyco-ops merged 1 commit from issue-504 into main 2026-07-14 00:53:26 +00:00
Conversation 2 Commits 1 Files changed 5 +54 -45
coilyco-ops commented 2026-07-13 21:57:36 +00:00
Member
Copy link

This changes the warded dev-base git identity path so commits from warded containers stamp the coilyco-ops bot identity instead of an example fallback. The self-name still drives the status line and session banner, but git committer attribution now resolves to the deployment bot.

  • Updates the baked agent-name.sh gitidentity hook to write the bot committer name and email, with explicit override envs for a future ward source of truth.
  • Refreshes the docs so they describe the split between display self-name and git identity.
  • Adds regression coverage for the commit identity defaults and the checked-in ward agents bundle.

closes #504
ward.workflow: pull-request-and-merge

Validation:

  • ward doctor
  • ward exec test -- tests/test_agent_name_gitidentity.py tests/test_ward_specs_bundle.py
  • ward exec pre-commit-all
This changes the warded dev-base git identity path so commits from warded containers stamp the coilyco-ops bot identity instead of an example fallback. The self-name still drives the status line and session banner, but git committer attribution now resolves to the deployment bot. - Updates the baked `agent-name.sh` gitidentity hook to write the bot committer name and email, with explicit override envs for a future ward source of truth. - Refreshes the docs so they describe the split between display self-name and git identity. - Adds regression coverage for the commit identity defaults and the checked-in ward agents bundle. closes #504 ward.workflow: pull-request-and-merge Validation: - `ward doctor` - `ward exec test -- tests/test_agent_name_gitidentity.py tests/test_ward_specs_bundle.py` - `ward exec pre-commit-all`
fix: stamp ward git identity as coilyco-ops
All checks were successful
ci / ward-doctor (pull_request) Successful in 17s
Details
ci / gate (pull_request) Successful in 55s
Details
d782354f33
coilyco-ops force-pushed issue-504 from d782354f33
All checks were successful
ci / ward-doctor (pull_request) Successful in 17s
Details
ci / gate (pull_request) Successful in 55s
Details
to dca6d7e7d6
All checks were successful
ci / ward-doctor (pull_request) Successful in 3m20s
Details
ci / gate (pull_request) Successful in 3m56s
Details
2026-07-14 00:47:28 +00:00 Compare
coilyco-ops commented 2026-07-14 00:52:41 +00:00
Author
Member
Copy link

WARDED_WORKFLOW: #508

details workflow: pull-request-and-merge; review summary: skipped intentionally because the temporary ward default pending brokered QA. felt: the rebase was clean once the deleted test file was dropped. confidence: high. surprises: `ward doctor` on the local checkout lagged behind the live PR status until CI reran. follow-ups: none.
WARDED_WORKFLOW: https://forgejo.coilysiren.me/coilyco-flight-deck/agentic-os/pulls/508 <details><summary>details</summary> workflow: pull-request-and-merge; review summary: skipped intentionally because the temporary ward default pending brokered QA. felt: the rebase was clean once the deleted test file was dropped. confidence: high. surprises: `ward doctor` on the local checkout lagged behind the live PR status until CI reran. follow-ups: none. </details>
coilyco-ops commented 2026-07-14 00:53:16 +00:00
Author
Member
Copy link

WARDED_WORKFLOW review-note

verdict: pass
confidence: high
summary: PR 508 now directly stamps the warded dev-base git committer identity as coilyco-ops <coilyco-ops@coilysiren.me> while keeping self-name display behavior separate. The patch updates the docs and regression tests for the bot identity path, and it keeps WARD_GIT_NAME / WARD_GIT_EMAIL overrides for future ward-owned sources.
findings: []
conclusion: The director review passes. Forgejo CI is green and ward exec test -- tests/test_agent_name_gitidentity.py passed locally against the PR head, so the PR is ready for the configured merge lane.

WARDED_WORKFLOW review-note verdict: pass confidence: high summary: PR 508 now directly stamps the warded dev-base git committer identity as `coilyco-ops <coilyco-ops@coilysiren.me>` while keeping self-name display behavior separate. The patch updates the docs and regression tests for the bot identity path, and it keeps `WARD_GIT_NAME` / `WARD_GIT_EMAIL` overrides for future ward-owned sources. findings: [] conclusion: The director review passes. Forgejo CI is green and `ward exec test -- tests/test_agent_name_gitidentity.py` passed locally against the PR head, so the PR is ready for the configured merge lane.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
burndown-2026-06
Backlog burndown June 2026

  
coherence-core
Core review set for the warded control plane coherence milestone. These issues form the release spine; adjacent milestone issues are stretch or supporting work.

  
consult
A human decision, design, or external action must happen first. Fail-closed default for unlabeled/low-confidence.

  
headless
Agent can take it from open issue to merged change with no human in the loop. The auto-burndown queue.

  
interactive
Agent does the work but pauses at a human checkpoint mid-flight (design choice, destructive step, human-only verification).

  
P0
now / blocking / urgent

  
P1
next / important

  
P2
backlog / will do

  
P3
someday / low

  
P4
icebox / frozen / reopen if it becomes real

No labels
burndown-2026-06
coherence-core
consult
headless
interactive
P0
P1
P2
P3
P4
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
coilyco-flight-deck/agentic-os!508
No description provided.