docs: share project instructions and mirror agent skills - #32
Merged
Conversation
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.
Project agent audit — 2026-09-06
Scope and findings
Audited tracked root instructions, both local standards, contributor/security/release
instructions, the historical GOAL, root SKILL.md, Makefile, CI, and release packaging.
Excluded private reference data, dependencies, build output, and nested worktrees.
The original checkout is preserved; this branch starts at origin/main 7857392.
The existing setup-documentation PR #26 and dependency PRs are separate work and remain untouched.
Changes and reconciliation
Preserved the full agent CLI contract in docs/MACHINE_CONTRACT.md and contributor
guidance in docs/PROJECT_INSTRUCTIONS.md. Identical root pointers and Cursor's
alwaysApply project rule load the same shared operating and project guidance.
Applied authorized-action persistence, material-only clarification, concrete approval,
explicit boundaries, scoped verification, artifact proof, and honest blocker/skip reporting
from the model guide: https://developers.openai.com/api/docs/guides/latest-model?model=gpt-6-astra.
The root published SKILL.md is the only existing skill package. Copied it byte-for-byte
to all three harnesses (one file each), preserving its permissions and product contract.
There were no competing copies, helpers, or provider settings to reconcile. Keep the
root publication artifact and mirrors synchronized on future edits.
Reused OpenLoop PR #176's checker and 16 regression cases; generalized its introduction
and remote-default fallback only. make check-agents participates in make check and CI.
It covers missing/deleted/untracked files, drift, identical foreign references in both
slash forms, CRLF preservation, source-preserving repair, and ambiguity refusal.
Retained both .claude standards at their established paths as shared local documents;
all harnesses explicitly read them. CLI_CONVENTIONS.md remains byte-identical to its
sibling copy; it requires no read from another checkout. Its historical @import
description now routes through the root pointer to explicit shared-document reads.
Preserved frozen API/JSON semantics, credential privacy, immutable security tests,
no-doctor decision, GM1-only guarantee, Conventional Commits, and release/publish gates.
Fixed the release-playbook filename's case in the relocated contributor document.
Included the pointer targets and referenced policy docs in GoReleaser archives, and
the SKILL.md already promised by its footer. Publishing triggers are unchanged.
The required make check formatter adjusted two Go call layouts in config.go and
config_test.go; no runtime logic or test assertion changed.
Validation
make check passed: mirror parity, all 16 checker regression cases, tidy, formatting,
vet, golangci-lint (0 issues), and race tests (7 passing packages; 1 has no tests).
go build ./... and make build passed; the built binary exists and is nonempty.
Root pointers are byte-identical; root SKILL.md matches all three skill copies.
goreleaser check validated the configuration. Final diff and whitespace reviewed.
Skipped and limitations
No live account login, workout retrieval, program creation, ClawHub publication,
release, deployment, or full cross-platform archive build was performed. Runtime and
account behavior is unchanged; fixture/race tests cover the existing security guards.
Windows Unix-permission assertions remain platform-inapplicable where the existing
suite says so; Linux CI runs them. Setup/credential ordering in the unchanged published
skill remains tracked by existing PR #26; this audit does not duplicate its rewrite.
Final CI follow-up
Linux build/race tests passed. CI lint initially failed because stable advanced to
Go 1.27.1 while the existing v2.12.2 linter was built with Go 1.26 and could not
parse the new standard library. Pinned only the lint lane to 1.26.x, matching the
locally verified Go 1.26.4/linter v2.12.2 pair. Build/race tests still use stable;
all lint rules and guards are unchanged. Updated the release playbook to keep
the pair explicit. This fixes the check environment without suppressing a finding.
Verified PR head
fa5d06ca293e25f11c3e9106ff03d09b80aa1afa— local committed tree and remote PR head match. harness skills in sync (1 files x 3 harnesses).The validation and follow-up evidence above apply to this head; unchanged product
paths retain their earlier checks. No merge, release, or manual deployment performed.
CI build/race and lint jobs both passed after pairing Go 1.26.x with linter v2.12.2.