Skip to content

docs: add CONTRIBUTING and CODE_OF_CONDUCT, file the #277 design note - #340

Merged
janekbaraniewski merged 1 commit into
mainfrom
chore/community-docs
Aug 31, 2026
Merged

docs: add CONTRIBUTING and CODE_OF_CONDUCT, file the #277 design note#340
janekbaraniewski merged 1 commit into
mainfrom
chore/community-docs

Conversation

@janekbaraniewski

Copy link
Copy Markdown
Owner

Housekeeping found while triaging the backlog — three things had been written but never committed.

CONTRIBUTING.md / CODE_OF_CONDUCT.md

Both existed in the working tree but were never added, so the repo shipped neither. GitHub surfaces them in the issue and PR flow (and links them from the community profile), and OpenSSF Scorecard — which this repo already runs on a schedule — checks for both.

Content is unchanged from what was drafted: CONTRIBUTING covers CGO/local setup, the test and lint commands, the import/style conventions the code actually follows, and a PR checklist; CODE_OF_CONDUCT is a Contributor-Covenant-derived policy with a reporting address.

docs/proposals/provider-options-and-gh-hosts.md

Files the existing local design note for issue #277 (Copilot against a GitHub Enterprise host) so the reporter and anyone else can see the intended shape before it is built. It proposes a general persisted AccountConfig.Options bag plus self-describing ProviderSpec.Options, then uses it to thread GH_HOST through a single ghCLI choke point — and flags the pre-existing unkeyed copilot apiCache as a correctness bug that must ship alongside, since one Provider instance serves all accounts.

Filing it as a proposal, not implementing it here. I've linked it from #277 for feedback on the two open questions (gh_host naming, and confirming GH_HOST covers gh api token resolution on GHE) before writing code.

CLAUDE.md provider section

It claimed "16 providers" while AllProviders() holds 35, and omitted the browser-session pattern (perplexity, opencode) entirely. Rewritten to point at AllProviders() as the authoritative list so the count cannot drift again, with the browser-session pattern added — including the rule not to refresh from the live browser cookie store on a poll, which is what clobbers a sibling account sharing the cookie domain (the bug #300 fixes).

Docs

This PR is the docs change. No code touched, so no docs/site/ page is affected.

🤖 Generated with Claude Code

https://claude.ai/code/session_01PNSGtKppEApHmGH9hD4BN1

The contributing guide and code of conduct had been written but never
committed, so the repo shipped neither. GitHub surfaces both in the issue
and PR flow, and OpenSSF Scorecard checks for them.

Also files the previously local design note for provider Options plus
multi-host Copilot support (issue #277), and refreshes the provider
section of CLAUDE.md: it claimed 16 providers when the registry holds 35,
and omitted the browser-session pattern entirely. Point at
AllProviders() as the authoritative list so the count cannot go stale
again.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PNSGtKppEApHmGH9hD4BN1
@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@janekbaraniewski
janekbaraniewski merged commit 0e09b53 into main Aug 31, 2026
17 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