coilyco-flight-deck/umbra

Watch

docs(agents): state the temp-clone expectation #279

Closed

coilyco-ops wants to merge 0 commits from claude/agents-temp-clone-note into main

coilyco-ops commented

2026-08-12 09:30:54 +00:00

Member

cli-guard is not in Agent Compose's repository-plan.yaml, so it has no resident checkout under ~/projects/<owner>/. Adds the same ## Checkout residency section landed across the other out-of-plan repos.

Two things came with it:

AGENTS.md was 380 chars over this repo's cap once the section landed. The Release section was the largest sink and was also wrong: it said a push to main fires release.yml. It does not. release.yml runs on release pushes only, and promote.yml gates main and fast-forwards release. Corrected to match the workflows.

docs/opcore-inline.md was 85 lines against an 80-line cap before this branch, so the repo could not pass its own documentation-layout hook. Reflowed the wrapped bullets to fuller width. No words dropped.

Docs only. No Go source, config, or workflow change. go vet ./... clean.

Merging fast-forwards release through promote.yml and cuts a version bump plus tap and bucket updates. Kai authorized that explicitly.

Not cleared here: this repo's local pre-commit still fails code-comments across http/opcore, http/specgen, and http/specverb, all pre-existing. The CI gate runs Go checks only, so this does not affect it. Tracked separately.

Refs coilysiren/inbox#349.

cli-guard is not in Agent Compose's `repository-plan.yaml`, so it has no resident checkout under `~/projects/<owner>/`. Adds the same `## Checkout residency` section landed across the other out-of-plan repos. Two things came with it: **AGENTS.md was 380 chars over this repo's cap** once the section landed. The Release section was the largest sink and was also **wrong**: it said a push to `main` fires `release.yml`. It does not. `release.yml` runs on `release` pushes only, and `promote.yml` gates `main` and fast-forwards `release`. Corrected to match the workflows. **docs/opcore-inline.md was 85 lines against an 80-line cap** before this branch, so the repo could not pass its own documentation-layout hook. Reflowed the wrapped bullets to fuller width. No words dropped. Docs only. No Go source, config, or workflow change. `go vet ./...` clean. Merging fast-forwards `release` through promote.yml and cuts a version bump plus tap and bucket updates. Kai authorized that explicitly. Not cleared here: this repo's local pre-commit still fails `code-comments` across `http/opcore`, `http/specgen`, and `http/specverb`, all pre-existing. The CI gate runs Go checks only, so this does not affect it. Tracked separately. Refs coilysiren/inbox#349.

coilyco-ops added 1 commit

2026-08-12 09:30:55 +00:00

docs(agents): state the temp-clone expectation

ci / test (pull_request) Successful in 48s

Details

ci / secrets (pull_request) Successful in 6s

Details

ci / lint (pull_request) Successful in 31s

Details

ci / lint (push) Successful in 34s

Details

ci / test (push) Successful in 48s

Details

ci / secrets (push) Successful in 5s

Details

promote / gate-and-promote (push) Successful in 1m1s

Details

release / release (push) Successful in 1m11s

Details

d9426dc3ae

cli-guard is not in Agent Compose's repository-plan.yaml, so it has no resident
checkout. Say so where an agent will read it.

AGENTS.md was 380 chars over this repo's 4000-char cap once the section landed,
so this also tightens it. The Release section was the largest sink and was also
wrong: it said a push to `main` fires release.yml, but release.yml runs on
`release` pushes only. promote.yml gates main and fast-forwards `release`, and
stage 2 cuts the tag from there. Corrected to match the workflows.

docs/opcore-inline.md was 85 lines against an 80-line cap before this branch.
Reflowed its wrapped bullets to fuller width, dropping no words.

Not cleared: this repo's gate still fails code-comments across http/opcore,
http/specgen, and http/specverb, all pre-existing and untouched here.

Refs coilysiren/inbox#349.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-authored-by: Kai Siren <coilysiren@gmail.com>
Co-authored-by: Claude <noreply@anthropic.com>

coilyco-ops requested review from coilysiren

2026-08-12 09:30:55 +00:00

coilyco-ops commented

2026-08-12 09:34:26 +00:00

Author

Member

Landed on main as d9426dc by fast-forward, so Forgejo did not auto-close this. Closing manually. The branch content is identical to what was reviewed here.

go vet ./... clean before the merge. The CI gate runs Go checks only (vet, build, race test, godoc-current, mod tidy, golangci-lint, secret scan), so the pre-existing local code-comments failures do not touch it.

Landed on `main` as `d9426dc` by fast-forward, so Forgejo did not auto-close this. Closing manually. The branch content is identical to what was reviewed here. `go vet ./...` clean before the merge. The CI gate runs Go checks only (vet, build, race test, godoc-current, mod tidy, golangci-lint, secret scan), so the pre-existing local `code-comments` failures do not touch it.

coilyco-ops closed this pull request

2026-08-12 09:34:31 +00:00

coilyco-ops referenced this pull request

2026-08-12 09:38:15 +00:00

Clear the code-comments violations blocking a green local gate #280

ci / test (pull_request) Successful in 48s

Details

ci / secrets (pull_request) Successful in 6s

Details

ci / lint (pull_request) Successful in 31s

Details

ci / lint (push) Successful in 34s

Details

ci / test (push) Successful in 48s

Details

ci / secrets (push) Successful in 5s

Details

promote / gate-and-promote (push) Successful in 1m1s

Details

release / release (push) Successful in 1m11s

Details

Pull request closed

Please reopen this pull request to perform a merge.

No reviewers

No milestone

No project

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/umbra!279

No description provided.

Rows
Columns