docs(agents): consolidate workflow notes from draft PRs#145
Conversation
Consolidates the accurate, verified additions from Tembo draft PRs #130, #131, #132, #137, #138, #139, #142, and #143 into one coherent AGENTS.md update: telemetry test isolation, convex-email component workflow, Homebrew formula automation, live-check LIVE_SEND behavior, docs-site types:check, blog refresh workflow, and the PostHog release annotation. Generated-By: PostHog Code Task-Id: ec2538f2-5c80-4142-b4b3-b1a53394862e
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Greptile SummaryThis PR consolidates agent workflow notes in
Confidence Score: 5/5Documentation-only changes are safe to merge once the environment credential guidance is corrected. The touched file is limited to contributor guidance, and the identified issue is a narrow accuracy problem in the documented credential sources. AGENTS.md
What T-Rex did
Reviews (1): Last reviewed commit: "docs(agents): consolidate workflow notes..." | Re-trigger Greptile |
| noise to your diff. | ||
|
|
||
| When adding or changing a provider adapter, verify it against a live account. These scripts read provider credentials from `.env.local`: | ||
| When adding or changing a provider adapter, verify it against a live account. These scripts read provider credentials from the repo-root `.env.local` (or your shell): |
There was a problem hiding this comment.
Keep env guidance accurate
This sentence drops .env, but each listed live-check script calls config({ path: ".env.local" }) and then config(), so missing keys are also loaded from the repo-root .env unless already supplied by .env.local or the shell. Agents following this guidance would miss a supported credential source.
Consolidates the eight overlapping Tembo AGENTS.md drafts (#130, #131, #132, #137, #138, #139, #142, #143) into one coherent update. Every claim was verified against the repo at
origin/main(package.json,scripts/,.github/workflows/,packages/email-sdk/test-preload.ts+ bothbunfig.tomls,packages/convex-email,.changeset/config.json).What this adds
[test] preload(dualbunfig.tomlsync caveat), injected-override pattern for telemetry tests, and user-facing opt-outs.release:ci+pack:checkcoverage, build-first requirement for in-packagebun test, and the committed schema-derived_generatedstubs (codegen needs a live deployment, effectively maintainer-only).bun run homebrew:updateand the automated post-publishautomation/homebrew-formula-<version>PR fromrelease.yml.<PROVIDER>_LIVE_SEND=truesends a real message.bun run types:checkvalidation step and theblog-schedule.ymlrefresh workflow (VERCEL_DEPLOY_HOOK_URL).POSTHOG_PERSONAL_API_KEY).Per-draft disposition
homebrew:updatescript,release.ymlautomation, branch name).bun run checkbefore every PR, which contradicts this repo's Local Checks rule (repo-wideoxfmt --writereformats ~170 drifted files). Itsrelease:cicorrection is already on main in prose form._generatedstubs kept. In-package check commands amended:bun testinside the package needs@opencoredev/email-sdkbuilt first.release:ci/pack:checkcoverage kept. Dropped "regenerate and commit_generatedafter changingconvex.config.ts/schema.ts/functions" — the committed stubs are generic and derive types from the schema, andconvex codegenneeds a configured deployment.blog-schedule.ymlparagraph kept (verified against the workflow). Live-check section was mostly redundant with the existing Local Checks block; Homebrew bullet superseded by the dedicated section from #130.<PROVIDER>_LIVE_SEND=truekept (verified in all fourscripts/check-*-account.ts). Dropped "they load.env.localthen.env" — the scripts only load.env.local. Its Homebrew section omitted the release.yml automation, so #130's version won.telemetry?: booleanclient option, fumadocstypes:check, annotation step inrelease.yml).All eight drafts are closed in favor of this PR.
Created with PostHog Code