Skip to content

feat(lint): move owned source selection to Go - #835

Draft
tbhb-agent wants to merge 3 commits into
mainfrom
fix/804-20261026
Draft

tbhb-agent wants to merge 3 commits into
mainfrom
fix/804-20261026

Conversation

@tbhb-agent

@tbhb-agent tbhb-agent commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Move owned-source selection and lint dispatch to agentrepo lint-source-files. Keep the existing Python APM filter and a frozen shell fallback behind the capability probe.

Why

Owned additions beside deployed content must remain eligible without linting vendored content. Consumers with older binary pins need the existing selection behavior throughout the N-2 compatibility window.

Verification

AC Command Result
AC1 mise run test-hooks tests/tasks/exclusions.bats Passed tracked and untracked selection, repository ignores, and empty repositories.
AC2 mise run test-hooks tests/tasks/exclusions.bats Passed default exclusions alongside generated directories, individual paths, spaces, and trailing slashes.
AC3 mise run test-hooks tests/tasks/exclusions.bats Passed exact APM exclusions, directory records, owned additions, and local harness content with and without a lockfile.
AC4 mise run test-hooks tests/tasks/lint-source.bats Frozen-shell comparisons preserve argument boundaries for spaces and line breaks.
AC5 mise run test-hooks tests/tasks/lint-source.bats Empty selection succeeds without starting a gate.
AC6 mise run test-hooks tests/tasks/lint-source.bats Frozen-shell comparisons preserve mode options, forwarded arguments, prose patterns, and glob overrides.
AC7 mise run test-hooks tests/tasks/lint-source.bats Failed discovery with partial output and failed deployment filtering stop before gate execution.
AC8 mise run test Linux passes, including frozen-shell status, stream, and ordering comparisons. Native macOS and Windows execution remains pending CI.
AC9 mise run test-hooks tests/tasks/lint-source.bats Passed capable, old, absent, unsupported, consumer-selected, and operational-failure wrapper behavior.

mise run lint, mise run test, mise run cover-go, and mise run lint-apm passed. The replacement checks failed against the original dispatch. Docs: updated docs/shared-lint-exclusions.md to explain capability probing and the compatibility fallback.

Risk

Selection mistakes can hide owned violations or lint deployed content. Revert this change to restore shell dispatch.

Compatibility: none

Related

Refs #804. Native platform verification remains for CI before this draft becomes ready.

Owned source gates must keep Git selection rules and exact APM
exclusions without hiding additions beside deployed content. Keep the
existing filter and a frozen shell fallback so older consumer binaries
continue to work throughout the compatibility window. Probe capability
before execution and propagate operational failures without retrying.

Assisted-by: codex:gpt-6.1-sol
Signed-off-by: Tony Burns <tony@tonyburns.net>
@tbhb-agent tbhb-agent added change:feat PR adds or changes a feature (Conventional Commit type feat) provenance:agent-assisted PR contains commits authored with AI assistance (see Assisted-by trailers) labels Oct 26, 2026
Shell integration doesn't measure help and discovery failures in Go
coverage, yet consumers depend on them when probing capability.

Assisted-by: codex:gpt-6.1-sol
Signed-off-by: Tony Burns <tony@tonyburns.net>
The remote contains the signed commits, so updating this branch's
base can't replace its published history.

Assisted-by: codex:gpt-6.1-sol
Signed-off-by: Tony Burns <tony@tonyburns.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:feat PR adds or changes a feature (Conventional Commit type feat) provenance:agent-assisted PR contains commits authored with AI assistance (see Assisted-by trailers)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant