chore(community): contributor infrastructure for the public release - #453
Merged
Conversation
Contributor
Greptile SummaryThe PR adds contributor-facing policies, templates, and a required CI workflow for the public release.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains. Important Files Changed
Reviews (5): Last reviewed commit: "chore(community): drop the pre-release d..." | Re-trigger Greptile |
Leo310
force-pushed
the
chore/contributor-infra
branch
from
September 2, 2026 17:47
782c323 to
8ed98b6
Compare
- CI workflow on every PR and on pushes to main/dev: Biome format + lint checks, svelte-check (src + tests), unit tests, production build. Deliberately not `biome ci`, which also enforces import ordering that neither `bun run format` nor `bun run lint` applies. - Remove the broken console.log check workflow (npm ci, missing script). - Release workflow: drop the stale papa-ts injection, pin actions v4/v2, frozen lockfile. - CONTRIBUTING.md: ways to contribute, branch model (PRs target main), AI-assistance policy, provider and skill recipes, docs-site pointer. - CODE_OF_CONDUCT.md (Contributor Covenant 2.1), SECURITY.md, PR template. - Issue templates rewritten for v2: current settings paths, platform and provider fields, no hard-coded assignees; template chooser routes questions to Discussions. - README links to CONTRIBUTING; AGENTS.md gains a Contributing pointer and drops the mention of a de.json that no longer exists.
Leo310
force-pushed
the
chore/contributor-infra
branch
from
September 2, 2026 17:47
8ed98b6 to
243cc83
Compare
The site regrouped docs by feature; features/search and internals/architecture now 404. Point at the current search/graph/agents pages and their how-it-works subpages.
…plate CONTRIBUTING now states the post-release branch model plainly: PRs target main. The parenthetical about dev being merged and retired is obsolete once that merge happens, which is imminent. Blank issues stay disabled so user reports keep the version/platform/ console fields, but the three templates were all user-facing, leaving maintainer tracking issues (refactors, flaky tests, release chores) with nowhere to go. The Task template is a free-text body for exactly those.
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.
What
Repo hygiene for the moment the plugin goes public: a working CI gate, a human-facing CONTRIBUTING.md (with the AI-assistance policy), code of conduct, security policy, PR template, and issue templates that match v2.
Branch model as agreed: PRs target
mainafter the release (devis merged and retired). CONTRIBUTING says so, with a one-line note aboutdevthat can be deleted at release time.Changes
.github/workflows/ci.yml(new): on every PR and on pushes tomain/dev, run Biome format + lint checks,bun run check,bun run test,bun run build. Usesbiome format/biome lintin check mode rather thanbiome ci, becausebiome ciadditionally enforces import ordering that neitherbun run formatnorbun run lintapplies;devcurrently has 20+ files that would fail on that alone..github/workflows/logs.ymlremoved:npm ciplus acreate-check.jsthat no longer exists..github/workflows/release.yml: drop thepapa-tsinjection (no longer a dependency),actions/checkout@v4,setup-bun@v2,--frozen-lockfile.CONTRIBUTING.md: ways to contribute (including non-code: skills, agents, docs), dev setup, branch/PR expectations, the AI-assistance policy, provider and skill recipes, docs-site pointer.CODE_OF_CONDUCT.md: Contributor Covenant 2.1.SECURITY.md: what counts, how to report..github/PULL_REQUEST_TEMPLATE.md: testing + AI-assistance disclosure.config.ymlroutes questions to Discussions.de.json.Also done outside the diff: labels
providerandskillcreated; #324, #312, #288, #289 taggedgood first issue+help wanted; #324 and #447 taggedprovider.How I tested it
All four gates pass locally on this branch after
bun install --frozen-lockfile: check (0 errors), format/lint (clean), test (110 files, 1630 tests), build. Workflow and template YAML parse. No plugin code changed.Needs a maintainer (repo settings, not in the diff)
CI / Format, lint, types, tests, buildcheck; consider dropping required signed commits (blocks most first-time contributors) — the GitGuardian check can stay.AI assistance
Drafted with Claude Code; the maintainer reviewed every file.