Skip to content

Prepare the repository for open source - #143

Merged
joelachance merged 2 commits into
mainfrom
claude/open-source-project-steps-9465ff
Oct 7, 2026
Merged

joelachance merged 2 commits into
mainfrom
claude/open-source-project-steps-9465ff

Conversation

@joelachance

@joelachance joelachance commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Gets the repo ready to announce as an open-source project.

Removed

  • PostHog analytics, completely: the telemetry package, baked keys, build and release wiring. The review --client surface still labels gx Cloud review history. gx enhance --client is gone because it only fed analytics.
  • The vendored review-knowledge corpus and its indexer. Those are third-party documents under their publishers' terms. The source list now lives at docs/review-knowledge-sources.yaml.
  • CONTEXT.md, which described the old JJ-based design, and the unused docs-site stub.
  • Every /Users/joe path in code, tests, and docs.

Added

  • NOTICE with copyright to Joe LaChance, AGPL-3.0-only.
  • CONTRIBUTING, SECURITY, CODE_OF_CONDUCT, CODEOWNERS, and issue and PR templates.
  • README sections on privacy, review knowledge, building from source, and licensing.
  • ci.yml, which runs gofmt, vet, build, and test on Linux and macOS.

Fixed

  • TestCleanupStaleStacksMarksMissingBookmarkStacksClosed relied on the machine's git identity and failed on Linux runners. It now sets its own.

Merge this before anything else. The GX_POSTHOG_KEY and GX_POSTHOG_HOST repository variables are already deleted. Until this lands, the release workflow on main still expects them, and its bake check fails on the next push to main.

Follow-ups after merge

  • The pre-existing history still holds the corpus and old local paths. Only rewrite history if a rights holder asks.

🤖 Generated with Claude Code

Remove PostHog analytics entirely: the telemetry package, its baked build
keys, the ldflags and verify-bake checks, and the release workflow vars. The
review client surface it resolved moves into the CLI, where it still labels
gx Cloud review history.

Remove the vendored review-knowledge corpus and its indexing pipeline. The
documents are third-party material under their publishers' terms. The source
list stays at docs/review-knowledge-sources.yaml, which the language-tag
contract test now reads.

Add NOTICE with copyright to Joe LaChance under AGPL-3.0-only, plus
CONTRIBUTING, SECURITY, CODE_OF_CONDUCT, CODEOWNERS, issue and PR templates,
and README sections on privacy, review knowledge, building from source, and
licensing. Add a Go CI workflow on Linux and macOS.

Delete the outdated CONTEXT.md and the unused docs-site stub, and scrub
local /Users/joe paths from code, tests, and docs.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

gx: https://gx.run/r/Yiv-PoOruzSUMknebVoCHQ
@joelachance
joelachance force-pushed the claude/open-source-project-steps-9465ff branch from 0a83e44 to e05be3a Compare October 6, 2026 20:08
TestCleanupStaleStacksMarksMissingBookmarkStacksClosed committed in a temp
repo without setting user.name or user.email, so it borrowed the machine's
identity. Linux CI runners have none, and git refused the commit. The test
now configures the repo itself, as its sibling tests do. The full suite also
passes with user.useConfigOnly set and no global identity.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

gx: https://gx.run/r/omjZcELdYNxGRD04BV_leQ
@joelachance
joelachance merged commit b047e6e into main Oct 7, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant