Repository navigation
chore: gofmt the 13 files left unformatted at 98f2ccc (no behaviour change) - #81
Merged
Merged
Conversation
…hange) #80 kept gofmt -l clean on every file it changed and noted 13 repo-wide hits already present at 98f2ccc; Sam approved clearing them in their own PR (#80 decision 4). This is that PR: `gofmt` on the committed bytes of exactly those 13 files, nothing else. 40 insertions, 39 deletions; with whitespace ignored the only change is the blank `//` line gofmt puts before an indented block in a doc comment (occupancy_test.go). gofmt -l over the committed bytes of every .go file: 13 -> 0 (on a Windows autocrlf checkout, `gofmt -l .` also lists every CRLF file; the count here is taken on `git show :<file> | gofmt -l`). go build ./..., go vet ./..., go test ./... green, also with MOOS_INTEGRATION=1 against ffs0's 4.0.7. authored-by: urn:moos:agent:claude-cowork.hp-z440 / urn:moos:session:sam.z440-cowork-workspace / t342-gofmt Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
All changes are formatting-only, and all 13 files are gofmt-clean.
0 open findings
What changed in this PR
Applies gofmt to 13 previously unformatted Go files without changing behavior.
Changes:
- Normalizes alignment and whitespace.
- Reformats Go documentation lists.
| File | Description |
|---|---|
internal/fold/errors.go |
Aligns error declarations. |
internal/graph/node.go |
Aligns struct fields. |
internal/graph/state.go |
Formats a documentation list. |
internal/hdc/crosswalk.go |
Removes trailing whitespace. |
internal/hdc/fiber.go |
Removes trailing whitespace. |
internal/kernel/liveness_test.go |
Aligns a struct literal. |
internal/kernel/sweep.go |
Formats a documentation list. |
internal/mcp/protocol.go |
Aligns struct fields. |
internal/operad/occupancy.go |
Aligns constants. |
internal/operad/occupancy_test.go |
Formats a documentation block. |
internal/reactive/engine_test.go |
Aligns map entries. |
internal/reactive/predicate_test.go |
Aligns map entries. |
internal/transport/auth_test.go |
Aligns an inline comment. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This was referenced Oct 9, 2026
Merged
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
R2 (t342): gofmt the 13 files that were already unformatted at 98f2ccc. This is #80's decision 4 ("separate gofmt-only commit for the 13 pre-existing files ★ yes, as its own small PR"), approved by Sam with "all advice is approved". The hp-laptop lead assigned it to the Z440 seat on mtdc-lab#16.
No behaviour change.
gofmtoutput on the committed bytes of exactly these 13 files: 40 insertions, 39 deletions.//line gofmt inserts before an indented block in a doc comment (internal/operad/occupancy_test.go).Files:
internal/fold/errors.gointernal/graph/node.go,internal/graph/state.gointernal/hdc/crosswalk.go,internal/hdc/fiber.gointernal/kernel/liveness_test.go,internal/kernel/sweep.gointernal/mcp/protocol.gointernal/operad/occupancy.go,internal/operad/occupancy_test.gointernal/reactive/engine_test.go,internal/reactive/predicate_test.gointernal/transport/auth_test.goChecks (Z440):
gofmt -l, over the committed bytes of every.gofilego build ./...go vet ./...go test ./...MOOS_INTEGRATION=1andMOOS_ONTOLOGY_PATHset to ffs0's 4.0.7The gofmt count is taken with
git show :<file> | gofmt -l. On a Windows autocrlf checkout,gofmt -l .also lists every CRLF file (90 here), which is checkout noise, not repo content.Built from
master23ba9ce, in its own worktree.authored-by: urn:moos:agent:claude-cowork.hp-z440 / urn:moos:session:sam.z440-cowork-workspace / t342-gofmt
🤖 Generated with Claude Code