ℹ️ About Codex in GitHub

\\n
\\n\\n[Your team has set up Codex to review pull requests in this repo](https://chatgpt.com/codex/cloud/settings/general). Reviews are triggered when you\\n- Open a pull request for review\\n- Mark a draft as ready\\n- Comment \\\"@codex review\\\".\\n\\nIf Codex has suggestions, it will comment; otherwise it will react with 👍.\\n\\n\\n\\n\\nCodex can also answer questions or update the PR. Try commenting \\\"@codex address that feedback\\\".\\n \\n

\",\"path\":null,\"side\":null,\"line\":null}","text_elements",[],{"_372":531,"_353":354,"_374":360},{"_375":532,"_377":-5,"_42":-5,"_378":-5,"_379":-5,"_96":306,"_92":381},[533],{"_46":527,"_92":534},"input_text",{"_372":536,"_353":354,"_374":360},{"_375":537,"_377":-5,"_42":-5,"_378":-5,"_379":-5,"_96":306,"_92":381},[538],{"_46":539,"_92":534},"\n /workspace/first-class-errors\n bash\n 2026-07-11\n Etc/UTC\n",{"_372":541,"_353":354,"_374":360},{"_375":542,"_377":-5,"_42":-5,"_378":-5,"_379":-5,"_96":543,"_92":381},[544,545],"developer",{"_46":547,"_92":534},{"_46":546,"_92":534},"\n## Skills\nA skill is a set of local instructions to follow that is stored in a `SKILL.md` file. Below is the list of skills that can be used. Each entry includes a name, description, and file path so you can open the source for full instructions when using a specific skill.\n### Available skills\n- imagegen: Generate or edit raster images when the task benefits from AI-created bitmap visuals such as photos, illustrations, textures, sprites, mockups, or transparent-background cutouts. Use when Codex should create a brand-new image, transform an existing image, or derive visual variants from references, and the output should be a bitmap asset rather than repo-native code or vector. Do not use when the task is better handled by editing existing SVG/vector/code-native assets, extending an established icon or logo system, or building the visual directly in HTML/CSS/canvas. (file: /opt/codex/skills/.system/imagegen/SKILL.md)\n- openai-docs: Use when the user asks how to build with OpenAI products or APIs and needs up-to-date official documentation with citations, help choosing the latest model for a use case, or model upgrade and prompt-upgrade guidance; prioritize OpenAI docs MCP tools, use bundled references only as helper context, and restrict any fallback browsing to official OpenAI domains. (file: /opt/codex/skills/.system/openai-docs/SKILL.md)\n- plugin-creator: Create and scaffold plugin directories for Codex with a required `.codex-plugin/plugin.json`, optional plugin folders/files, and baseline placeholders you can edit before publishing or testing. Use when Codex needs to create a new local plugin, add optional plugin structure, or generate or update repo-root `.agents/plugins/marketplace.json` entries for plugin ordering and availability metadata. (file: /opt/codex/skills/.system/plugin-creator/SKILL.md)\n- skill-creator: Guide for creating effective skills. This skill should be used when users want to create a new skill (or update an existing skill) that extends Codex's capabilities with specialized knowledge, workflows, or tool integrations. (file: /opt/codex/skills/.system/skill-creator/SKILL.md)\n- skill-installer: Install Codex skills into $CODEX_HOME/skills from a curated list or a GitHub repo path. Use when a user asks to list installable skills, install a curated skill, or install a skill from another repo (including private repos). (file: /opt/codex/skills/.system/skill-installer/SKILL.md)\n### How to use skills\n- Discovery: The list above is the skills available in this session (name + description + file path). Skill bodies live on disk at the listed paths.\n- Trigger rules: If the user names a skill (with `$SkillName` or plain text) OR the task clearly matches a skill's description shown above, you must use that skill for that turn. Multiple mentions mean use them all. Do not carry skills across turns unless re-mentioned.\n- Missing/blocked: If a named skill isn't in the list or the path can't be read, say so briefly and continue with the best fallback.\n- How to use a skill (progressive disclosure):\n 1) After deciding to use a skill, open its `SKILL.md`. Read only enough to follow the workflow.\n 2) When `SKILL.md` references relative paths (e.g., `scripts/foo.py`), resolve them relative to the skill directory listed above first, and only consider other paths if needed.\n 3) If `SKILL.md` points to extra folders such as `references/`, load only the specific files needed for the request; don't bulk-load everything.\n 4) If `scripts/` exist, prefer running or patching them instead of retyping large code blocks.\n 5) If `assets/` or templates exist, reuse them instead of recreating from scratch.\n- Coordination and sequencing:\n - If multiple skills apply, choose the minimal set that covers the request and state the order you'll use them.\n - Announce which skill(s) you're using and why (one short line). If you skip an obvious skill, say why.\n- Context hygiene:\n - Keep context small: summarize long sections instead of pasting them; only load extra files when needed.\n - Avoid deep reference-chasing: prefer opening only files directly linked from `SKILL.md` unless you're blocked.\n - When variants exist (frameworks, providers, domains), pick only the relevant reference file(s) and note that choice.\n- Safety and fallback: If a skill can't be applied cleanly (missing files, unclear instructions), state the issue, pick the next-best approach, and continue.\n","\nFilesystem sandboxing defines which files can be read or written. `sandbox_mode` is `danger-full-access`: No filesystem sandboxing - all commands are permitted. Network access is enabled.\nApproval policy is currently never. Do not provide the `sandbox_permissions` for any reason, commands will be rejected.\n","turn/started",{"_353":354,"_87":550},{"_103":-5,"_42":360,"_361":551,"_363":364,"_365":552},[],"inProgress",{"_365":554,"_353":354},{"_555":556,"_92":557},"activeFlags",[],"active","thread/started",{"_560":561},"thread",{"_562":-5,"_563":-5,"_564":565,"_566":364,"_417":418,"_567":15,"_568":-5,"_569":-5,"_42":354,"_570":571,"_181":-5,"_572":-5,"_161":573,"_574":199,"_575":-5,"_422":576,"_365":577,"_578":-5,"_77":579,"_580":364},"agentNickname","agentRole","cliVersion","0.128.0","createdAt","ephemeral","forkedFromId","gitInfo","modelProvider","openai","parentThreadId","/opt/codex/sessions/2026/07/11/rollout-2026-07-11T09-57-50-019f509c-a461-77b3-b734-d9f57713e8e5.jsonl","preview","recencyAt","vscode",{"_92":369},"threadSource",[],"updatedAt",[],"messages",[584],{"_42":585,"_586":587,"_588":589,"_590":589,"_375":591,"_365":592,"_377":-5,"_593":32,"_594":595,"_596":597,"_598":-5},"e8b8307a-a20f-4932-9c67-72064c08a341","author",{"_96":306,"_181":-5,"_594":607},"create_time",1783763869.976775,"update_time",{"_603":46,"_604":605},"finished_successfully","weight","metadata",{"_599":600},"recipient","all","channel","l1239dk1",{"_601":602},"inline_comments",[],"content_type","parts",[606],"@codex Ok for the fix ?\ndocs: add a branch-management policy\n\n## Summary\n\nAdd a `Branches` section to `CONTRIBUTING.md`, defining how a branch is named, created, scoped, and retired. It completes the workflow the guide already documents for commits and pull request titles, so the guide now reads in order: branch → commit → request.\n\n## Type of change\n\n* [ ] Bug fix\n* [ ] New feature\n* [ ] Breaking change\n* [ ] Refactoring\n* [ ] Analyzer / diagnostic change\n* [ ] Tests\n* [x] Documentation\n* [ ] Build / CI / tooling\n\n## Changes\n\n* Add a `## Branches` section to `CONTRIBUTING.md`, placed before `## Commit messages` so the guide reads in workflow order (branch → commit → request).\n* Adopt `/` branch names, where `` is the owner's GitHub handle (`sylvain/…`, `claude/…`, `dependabot/…`). The change type stays on the commits, where the hook and CI enforce it; a branch carries no type, so `claude/` and `dependabot/` are the rule, not an exception.\n* Fix the branch as the disposable workspace of one pull request: one branch ↔ one pull request, cut from a fresh `origin/main`, never revived once merged or closed.\n* Document staying current (rebase while private, merge once shared), the ban on rewriting a branch under review, and a pre-PR scope check (`git log`/`git diff` against `origin/main`).\n* Follow the existing section style — Why, the rule, the doctrine, examples, anti-patterns.\n\n## Testing\n\nDocumentation-only change; no source, build, or test target is affected, so the `dotnet` commands below were not run.\n\n* [ ] `dotnet build FirstClassErrors.sln`\n* [ ] `dotnet test FirstClassErrors.sln`\n* [ ] Analyzer tests pass (`FirstClassErrors.Analyzers.UnitTests`)\n\nThe commit message was validated against the repository's own linter, `tools/commit-lint/lint-commit-message.sh`, in both local and `--ci` modes.\n\n## Documentation\n\nThis PR *is* the documentation change (`CONTRIBUTING.md`, a repository-governance document). None of the boxes below apply: no public API / error documentation, `README`, or `doc/` update is involved, and no French translation is needed — `CONTRIBUTING.md` is English-canonical with no French counterpart, and no user-facing library behavior changed.\n\n* [ ] Public API / error documentation updated\n* [ ] README / `doc/` updated\n* [ ] French translation (`doc/README.fr.md`) updated if user-facing behavior changed\n* [ ] No documentation change required\n\n## Related issues\n\nNone — the policy was proposed and refined directly; there is no tracking issue.\n\n---\n🤖 Generated with [Claude Code](https://claude.com/claude-code)\n\nhttps://claude.ai/code/session_01MAHFZWHiPnWKLfA6BiGLDK\n\n---\n_Generated by [Claude Code](https://claude.ai/code/session_01MAHFZWHiPnWKLfA6BiGLDK)_\n**_{_{![P2 Badge](https://img.shields.io/badge/P2-yellow?style=flat)}} Leave a path to fix rejected commit messages**\n\nIn PRs where commit-lint CI or the reviewer rejects a commit message, this new MUST NOT blocks the documented remediation path: the existing guide says review refuses non-conforming messages and lets the author correct them before merge, and the CI failure text explicitly tells authors to use `git rebase -i`. Because commit messages cannot be fixed by adding a follow-up commit, a branch that is already under review becomes impossible to bring into compliance unless this rule allows review-requested history fixes while the branch is not shared with collaborators.\n\nUseful? React with 👍 / 👎.\n**_{_{![P2 Badge](https://img.shields.io/badge/P2-yellow?style=flat)}} Permit generated Dependabot branch names**\n\nWith the current `.github/dependabot.yml`, Dependabot branch names are not customized, and GitHub documents the default as `dependabot/PACKAGE_MANAGER/DEPENDENCY` (for example `dependabot/nuget/...`), not the two-component `/` shape required here. That makes routine NuGet and GitHub Actions Dependabot PRs non-conforming even though the new text says `dependabot/…` is part of the rule; either the policy needs to allow Dependabot's generated shape or the Dependabot config needs to be changed.\n\nUseful? React with 👍 / 👎.\n\n### 💡 Codex Review\n\nHere are some automated review suggestions for this pull request.\n\n**Reviewed commit:** `8cc6bfcab7`\n \n\n

ℹ️ About Codex in GitHub

\n
\n\n[Your team has set up Codex to review pull requests in this repo](https://chatgpt.com/codex/cloud/settings/general). Reviews are triggered when you\n- Open a pull request for review\n- Mark a draft as ready\n- Comment \"@codex review\".\n\nIf Codex has suggestions, it will comment; otherwise it will react with 👍.\n\n\n\n\nCodex can also answer questions or update the PR. Try commenting \"@codex address that feedback\".\n \n

",{},"created",1783763900.0133657,"Completing the task","event_type","info",{"_92":381,"_96":93,"_375":1011},{"_92":615,"_616":617},"partial_repo_snapshot","files",[618],{"_161":456,"_619":-5,"_620":621,"_622":-5},"contents","line_range_contents",[623],"missing_reason",{"_624":309,"_625":626,"_375":627,"_628":59},"line_range_start","line_range_end",546,[629,199,630,631,632,199,633,199,634,635,636,637,199,638,639,199,640,199,641,642,199,643,644,643,199,645,646,647,648,199,649,650,199,651,199,652,199,653,654,655,656,657,658,199,643,659,660,661,662,643,199,663,664,665,199,666,199,667,668,669,670,671,672,673,199,674,675,676,674,677,678,679,680,681,682,683,684,685,686,687,688,689,690,691,692,693,694,695,696,697,698,699,700,701,702,703,704,705,706,707,708,709,710,711,712,713,199,674,675,714,715,674,199,716,717,199,718,199,719,720,721,199,722,723,724,725,726,727,199,728,729,730,199,731,732,733,199,734,735,736,199,737,738,739,199,740,199,741,742,743,744,745,746,199,747,199,748,742,749,750,751,752,753,754,755,756,757,199,758,199,759,760,761,762,199,652,199,763,764,199,643,765,766,767,643,199,768,199,643,769,770,771,643,199,772,773,774,775,199,666,199,776,777,778,779,199,780,199,643,781,199,782,199,783,643,199,784,785,786,787,788,789,790,199,791,199,792,793,199,794,795,796,797,798,799,800,801,802,803,804,805,806,199,807,808,199,809,199,810,199,811,742,812,813,814,815,816,199,817,818,819,199,820,821,822,823,824,825,826,199,827,828,829,830,199,643,831,643,199,832,199,833,834,835,836,837,838,839,840,841,199,842,199,843,844,845,199,846,847,848,199,849,199,850,851,852,853,199,854,855,856,857,858,199,859,860,861,862,863,864,199,865,866,867,868,199,869,199,870,871,199,643,872,199,873,874,875,643,199,876,877,878,199,879,880,881,882,199,883,199,884,885,199,643,886,199,887,643,199,888,889,890,891,199,718,199,892,893,894,895,199,896,897,898,899,900,199,901,902,903,904,199,905,906,907,199,908,909,199,910,911,912,199,913,914,915,199,916,917,918,919,920,199,921,922,923,924,199,740,199,925,199,643,926,199,927,643,199,928,199,643,929,199,930,931,932,199,933,643,199,934,935,199,643,936,643,199,937,199,643,872,199,873,874,875,199,938,643,199,747,199,939,199,940,742,941,942,943,944,945,946,947,948,949,950,951,952,199,953,199,954,955,956,199,957,958,199,959,960,961,962,963,199,964,199,759,965,199,966,199,967,968,969,970,971,199,972,973,974,975,976,199,666,199,977,978,979,980,981,982,983,984,985,986,987,988,989,990,991,992,993,994,995,996,997,998,999,1000,199,740,199,1001,742,1002,1003,1004,1005,199,747,199,1006,742,1007,1008,1009,1010],"contains_end_of_file","# Contributing to FirstClassErrors","FirstClassErrors treats errors as first-class, documented, diagnosable","concepts. The history of the repository should be as legible as the errors the","library produces. This guide defines how commits are written here.","## Building and testing","* Target framework: **.NET Standard 2.0**.","* Build: `dotnet build FirstClassErrors.sln`","* Test: `dotnet test FirstClassErrors.sln`","* Analyzer tests, when touching analyzers: `dotnet test FirstClassErrors.Analyzers.UnitTests`","See [`CLAUDE.md`](CLAUDE.md) for the project layout and the broader change","guidelines.","## Enabling the commit-message hook","A `commit-msg` hook checks every message against the convention below before it","is recorded. It is versioned under `.githooks/`; enable it once per clone:","```","git config core.hooksPath .githooks","The same check runs in CI on every pull request, so a bypassed hook","(`git commit --no-verify`) is caught before merge. Merge commits are exempt.","The check itself lives in `tools/commit-lint/lint-commit-message.sh`, shared by","the hook and CI so the two never diverge.","The hook lets `fixup!`, `squash!`, and `amend!` commits through so you can build","an autosquash rebase; CI rejects them, so squash them away before merge.","## Branches","### Why","A pull request is read against its branch. Two branches carry the same feature.","The first was cut from `origin/main` an hour ago and holds three commits — its","pull request diff *is* the feature, and nothing else. The second was cut from a","local `main` three weeks stale, then revived for a second idea once the first","had merged; its diff carries fifteen commits, twelve already on `main`, and the","reviewer cannot tell the request from the residue.","$ git log --oneline origin/main..HEAD # the second branch","a1b2c3d feat(core): the change the pull request is for","9f8e7d6 feat(gendoc): a renderer already merged, dragged along","...twelve more commits already on main...","The branch is not the work. It is the disposable workspace of **one** pull","request — cut fresh from the remote, used once, discarded on merge. Everything","below follows from that.","### The rule","* A branch carries **one pull request**, and that pull request carries one"," coherent unit of work. Work unrelated to that unit MUST take its own branch —"," the branch-level reading of *two intentions, two commits*.","* `main` is written **only** by merge. No commit lands on `main` directly; it"," moves when a reviewed pull request is merged.","* A branch MUST be cut from the **tip of `origin/main`**, freshly fetched —"," never from a local `main` that may lag, nor from another topic branch:"," ```"," git fetch origin"," git switch -c / origin/main","* A branch name MUST take the form `/`. The"," `` is the branch owner's GitHub handle — the person or the tool the"," work belongs to: `sylvain/…`, `claude/…`, `dependabot/…`. The"," `` MUST be English, lowercase, kebab-case, and name the"," change, not the file it touches: `sylvain/gendoc-invalid-culture`, never"," `sylvain/GenDoc.cs`.","* A tool that generates its own branches owns its namespace and keeps its"," native layout beneath it — `dependabot/nuget/Newtonsoft.Json-13.0.1`,"," `renovate/…`. The `/` form binds the branches a"," person or an agent cuts by hand; a generator's scheme is the generator's to"," define, and fighting it buys nothing.","* The branch name carries **no type**. The type is a property of each commit,"," checked there by the hook and by CI; a branch gathers commits of several"," types, and a single prefix would name one and hide the rest — the same reason"," a multi-intention pull request title takes no `type:` (see *Pull request"," titles* below). The owner is what the name adds, because the owner is what the"," commits do not carry.","* A branch lives exactly as long as its pull request stays **open**, and MAY be"," reused only for that same request — review fixes, changes asked for on the"," pull request.","* Once the pull request is **merged or closed**, the branch is finished. It MUST"," NOT be revived, not even for follow-up on the same topic: a merged pull"," request cannot describe new work, and a closed one was set aside. Follow-up is"," a new branch, cut fresh from `origin/main`.","* To carry `main`'s progress into an open branch: while the branch is yours"," alone, **rebase** it onto `origin/main`; once others may have based work on"," it, **merge** `origin/main` in instead. Either keeps the branch current"," without rewriting what a collaborator has already pulled.","* Rewriting a branch's history — a force-push, a `git rebase -i` — is fine"," while the branch is **yours alone**, and is how a commit message the lint or"," a reviewer rejected gets fixed, even mid-review: a rejected message cannot be"," corrected by a follow-up commit (see *Commit messages*). Once anyone else may"," have **based work on it**, its history MUST NOT be rewritten — a force-push"," discards what was built on top. Work that is not yours is not yours to"," rewrite or delete.","* Before opening the pull request, **read the branch** against a fresh"," `origin/main`:"," git log --oneline origin/main..HEAD # the commits the request adds"," git diff --stat origin/main...HEAD # the files it touches"," If either shows something the request is not about, the branch has drifted —"," split it before review, not after.","### The doctrine","**The branch is the unit of work in progress; the pull request is what it","becomes.** One branch, one pull request, one unit of work — the same one-to-one","the doctrine draws between the commit and its change.","**The name says who, the commits say what.** A branch owns a pull request that","may carry a feature, the refactor that prepared it, and its tests at once; no","single type names it honestly. The type lives on each commit, where the hook","enforces it. The branch name adds the one thing the commits omit — whose work it","is — so `claude/…` and `dependabot/…` are not exceptions but the rule itself,","read the same on a human or a machine.","**A branch is disposable.** Its history is preserved by the merge commit that","lands it; the ref itself is cut fresh and deleted on merge. Nothing of value","lives only on a branch.","**A merged branch is spent.** Reviving it stacks new work on settled history and","forks from a `main` that has moved. The reviewer pays the cost, reading the","residue as if it were the request.","**Cut from the remote, not the local.** A local `main` lags silently; a branch","cut from it drags that lag into every diff. `origin/main`, freshly fetched, is","the only base.","**Unrelated work is a new branch, not a passenger.** A branch that carries two","changes forces a pull request that can describe neither — the branch-level form","of the commit that carries two intentions.","### Examples","| Branch | Why it fits |","|---|---|","| `sylvain/add-html-renderer` | Owner and change, named plainly. The type it will carry lives in its commits. |","| `claude/gendoc-invalid-culture` | An agent's branch; the description names the zone, not `GenDoc.cs`. |","| `dependabot/nuget/Newtonsoft.Json-13.0.1` | A generator keeps its native layout under its `dependabot/` namespace. |","| `sylvain/security-policy` | The description alone carries the subject; the branch needs no type. |","### Anti-patterns","| Branch or move | What is wrong |","| a commit pushed straight to `main` | `main` moves only by merge. Even a one-line fix takes a branch and a pull request. |","| `patch-1`, `my-work`, `tmp` | No owner, and it names nothing. A branch name is read in the pull-request list; it MUST say who owns what. |","| `feat/add-html-renderer` | A type in the owner's slot. The type belongs on the commits; the branch prefix is the owner: `sylvain/add-html-renderer`. |","| `sylvain/GenDoc.cs` | Names a file. It should name the change: `sylvain/gendoc-invalid-culture`. |","| `sylvain/corrige-le-rendu` | Not English. |","| reviving a merged `claude/add-html-renderer` for a follow-up | A merged branch is spent. Cut the follow-up fresh from `origin/main`. |","| a branch cut from a three-week-old local `main` | The pull request diff fills with commits already on `main`. Fetch first; cut from `origin/main`. |","| one branch carrying a feature and an unrelated CI tweak | No single pull request describes both. Two branches, two requests. |","| force-pushing a branch others have built on | Rewrites shared history and discards the work pushed on top. Rewrite only while the branch is yours alone. |","## Commit messages","This section adapts the [Conventional Commits 1.0.0](https://www.conventionalcommits.org/en/v1.0.0/)","specification. The key words MUST, MUST NOT, SHOULD, and MAY are to be","interpreted as described in [BCP 14](https://www.rfc-editor.org/info/bcp14), and","only when they appear in capitals.","A release is prepared. One needs to know what a branch contains, what to carry","into it, and which version number comes out.","a3f1c2e fix bug","8b41d90 update renderer","1d0e4aa wip","This history teaches nothing. Every question forces a diff open.","a3f1c2e fix(gendoc): render error examples with the invariant culture","8b41d90 feat(gendoc): emit an RFC 9457 problem type in examples","1d0e4aa refactor(gendoc): extract output routing into a writer","This one answers three questions without opening a single diff: what the branch","contains, which commit to carry into a release, and whether the version moves","from `1.4.2` to `1.4.3` or to `1.5.0`. That is the reading of the reviewer, and","of whoever prepares the release. Tomorrow it will be a tool's.","The rule bears on **each commit**, not on a merge message. A commit travels","alone: it is cherry-picked onto a release branch, listed in a","`git log --oneline`, read in isolation six months later. Its message MUST stand","on its own.","#### Form","[()][!]: ","[body]","[footers]","* The commit MUST begin with a type, optionally followed by a scope and a `!`,"," then a colon and a space.","* Everything written in the message MUST be in English — header, body, and"," footers.","* A commit MUST carry a single type, that of its intention. Two independent"," intentions MUST be two commits: the message forces the split that ought to"," happen.","#### Types","Ordered here with the two version-driving types first, then the rest","alphabetically. The list is closed.","| Type | When to use | Minimal effect on the version |","|---|---|---|","| `feat` | A new capability, visible to the consumer of the package | `MINOR` |","| `fix` | The correction of a defective behaviour | `PATCH` |","| `build` | Build system, dependencies, packaging, deployment artefacts | none imposed |","| `chore` | What touches neither production code nor its delivery | none imposed |","| `ci` | Pipeline configuration | none imposed |","| `docs` | Documentation only | none imposed |","| `perf` | A performance gain, at constant observable behaviour | none imposed |","| `refactor` | Restructuring, at constant observable behaviour | none imposed |","| `revert` | The reversal of an earlier commit | per what it reverts |","| `style` | Formatting with no semantic effect | none imposed |","| `test` | Tests only | none imposed |","The type MUST be lowercase and belong to this table. A breaking change carried","by any of these types produces a `MAJOR`.","#### Scope","The scope MAY be provided. When present it MUST be lowercase and MUST be one of:","| Scope | Covers |","| `core` | `FirstClassErrors` — the runtime library (`Error`, `Outcome`, `ErrorCode`, `ErrorContextKey`, …) |","| `analyzers` | `FirstClassErrors.Analyzers` — the Roslyn analyzers and their `FCExxx` diagnostics |","| `cli` | `FirstClassErrors.Cli` — the command-line tool |","| `gendoc` | `FirstClassErrors.GenDoc` and its worker — the documentation generator |","| `testing` | `FirstClassErrors.Testing` — the test-support package |","This list lives here, in the repository, where a tool can check it. A scope","MUST NOT be a file name or a class name: those move; the zone they inhabit does","not. `fix(core):`, never `fix(ErrorCode.cs):`.","The scope is optional here because the two published packages","(`FirstClassErrors` and `FirstClassErrors.Testing`) share a single tag-driven","version: the scope carries no versioning weight, only readability. What does not","belong to a component takes **no** scope — repository infrastructure (the","solution, `Directory.Build.props`, the workflows, `.gitignore`, `CLAUDE.md`),","repository-wide documentation, ADRs, and the `FirstClassErrors.Usage` samples:","`ci: …`, `docs: …`.","When one atomic change crosses several components, the commit MUST carry all","their scopes, comma-separated with no space and ordered alphabetically. The","order is alphabetical so a given pair is always written the same way, and found","again with a single `git log --grep`.","fix(cli,gendoc): thread cancellation through the generate command","#### Description","* It MUST be in the imperative present: `add`, not `added` nor `adds`. The"," description completes one sentence — *If applied, this commit will …* — and"," only the imperative fits it: *…will add Outcome.Map*.","* It MUST begin with a lowercase letter and MUST NOT end with a period. The"," header line is not a sentence; it is a title.","* The full header line — type, optional scope, optional `!`, colon and"," description — MUST fit in 72 characters. Beyond that, once the abbreviated"," hash is prefixed, it overflows the 80 columns of a terminal in a"," `git log --oneline`.","#### Body","The body MAY be provided, after a blank line. It explains **why** the change","happens — the constraint, the symptom, the trade-off. The *what* is already in","the diff; repeating it is noise.","When that why is not readable from the diff, the body SHOULD be provided.","Abstaining is paid for six months later, on a commit no one can interpret any","more.","#### Footers","Footers MAY be provided, after a blank line. Each footer MUST take the form","`Token: value`. The token MUST be words separated by hyphens, **each word","capitalized**: `Co-Authored-By`, `Reviewed-By`, `Refs`, `Reverts`.","`BREAKING CHANGE` is the sole exception to this form.","> This \"every word capitalized\" casing is a deliberate departure from the usual","> single-initial convention. It exists so that hand-written footers stay","> consistent with the trailers this repository's automated commits already","> carry — `Co-Authored-By`, `Claude-Session`. One rule for every footer beats","> two.","When an issue exists, its number MUST live in a `Refs:` footer, and MUST NOT","appear in the description — the description states the change, not where it was","requested. The footer carries the key (`#142`), never the URL: a commit message","is not rewritten, and the number survives what an address does not. (A tooling","footer such as `Claude-Session` is a URL by nature, and is exempt from that last","point.)","A commit is **not** the place to close an issue. Closing is a","repository-workflow concern: put `Closes #142` in the pull request description,","and GitHub closes the issue on merge. The commit itself stays neutral, carrying","at most a `Refs:`.","#### Breaking changes","A breaking change MUST be signalled twice: by a `!` placed just before the","colon, and by a `BREAKING CHANGE:` footer in capitals.","feat(core)!: fail Outcome.To with an Outcome instead of throwing","BREAKING CHANGE: Outcome.To returns a failed Outcome where it","used to throw on a null conversion. Callers must handle the Outcome instead","of catching.","The `!` is what one sees in a `git log --oneline`. The footer is what one reads","when it is time to migrate. The two have different readers; neither replaces the","other.","What is breaking reads on the **published contract**, not on internal code. In","this repository that contract explicitly includes error codes, diagnostic IDs","(`FCExxx`), and public types: renaming any of them is a breaking change (see","`CLAUDE.md`). Renaming an `internal` type breaks nothing.","#### Reverts","A revert commit MUST carry the `revert` type, repeat the description of the","reverted commit, and reference its SHA in a `Reverts:` footer.","revert(gendoc): emit an RFC 9457 problem type in examples","Reverts: b36765a","A revert's effect on the version is qualified like any commit's: from the","consumer, on the published contract. Reverting a change not yet released","neutralizes its effect. Removing a capability already released is a breaking","change, and the commit MUST then carry the `!` and the `BREAKING CHANGE` footer.","**The issue is the unit of the request, the commit the unit of the change.** An","issue produces as many commits as it carries intentions: the feature, the","refactor that prepared it, the fix found in review. Each carries its own type,","all carry the same `Refs:`.","**The type is the intention, not the content of the diff.** A feature arrives","with its tests, its API documentation, its sample: the commit stays a `feat`.","`test` and `docs` designate a change that touches *only* tests, *only*","documentation. Splitting a `feat` into five commits because it spans five","directories manufactures commits that do not compile alone.","**`feat` or `fix` is decided from outside the component.** The criterion is not","the size of the diff, it is what the consumer of the package observes. Three","lines that restore the promised behaviour are a `fix`. One line that opens a new","capability is a `feat`.","**`refactor` and `perf` make a promise: observable behaviour does not change.** A","`refactor` that fixes a bug in passing is a mislabelled `fix` — and the","correction becomes invisible to whoever prepares the release.","**`chore` is not the bin.** Everything that fits nowhere lands there, and the","type ends up meaning nothing. Before writing `chore`, reread the table.","**What is breaking reads on the published contract**, not on internal code. A","renamed `internal` type breaks nothing. A changed return type, a renamed","serialized field, an error code, a diagnostic ID — those do.","**The wrong type is fixed before the merge.** A `git rebase -i` rewrites the","message while the commit has not reached a shared branch. After that, the cost","of the correction exceeds the cost of the error: leave it and move on.","**The version number is decided by reading the history.** Whoever prepares the","release reads the increment there: a lone `fix` gives a `PATCH`, a `feat`","imposes at least a `MINOR`, a breaking change imposes a `MAJOR`.","`FirstClassErrors` and `FirstClassErrors.Testing` share one tag-driven version,","so the highest increment across the release applies to both.","**Who decides what.** The author of the commit chooses the type and the scope.","The reviewer of the pull request refuses a non-conforming message as they refuse","non-conforming code. The maintainers own the list of scopes and the list of","types.","**A feature, with scope and issue.**","feat(analyzers): add FCE016 for an undocumented error code","Refs: #142","**A fix whose why is not readable from the diff.**","fix(gendoc): render error examples with the invariant culture","Sample amounts were formatted with the host's culture, so the Verify","baselines matched on an invariant machine and failed on a comma-decimal one.","CI and developers disagreed on the very same commit.","Refs: #128","**A refactor, promising nothing but iso-behaviour.** Neither body nor footer:","the diff says it all.","refactor(core): extract transience computation into TransienceCalculator","**A breaking change, with the migration instruction.**","Refs: #150","Ordered as the rules are: type, scope, description, body, breaking, issue.","| Message | What is wrong |","| `chore: handle a null error code` | A `fix` in disguise. The release preparer will not see it, and the version will not move when it should. |","| `feat: refactor the extraction reader` | The type contradicts the description. One of the two is lying. |","| `fix(core): correct transience and add a CI cache` | Two changes, two commits. No version can describe this one. |","| `fix(ErrorCode.cs): formatting` | The scope names a file. It designates a zone: `core`. |","| `feat(gendoc, cli): carry the source description` | A space after the comma, and the order is not alphabetical. Two spellings for one pair — write `feat(cli,gendoc):`. |","| `fix(core): Fixed the null dereference.` | Capital, past tense, trailing period. Three form rules broken, one useful word. |","| `feat(core): add support` | Support for what? The description must stand alone in a `git log`. |","| `fix(core): change line 42 of Error` | The description names a line. It should name a change. |","| `fix(gendoc): render with the invariant culture` — body: `Replaced DateTime.Now with CultureInfo.InvariantCulture` | The body repeats the diff. It should say why the culture varied across hosts. |","| `feat(core)!: fail Outcome.To with an Outcome` — no footer | The `!` warns; it migrates no one. |","| `feat(core): add Outcome.Map (#142)` | The issue eats the 72 characters of the description. Its place is a footer. |","| `refs: #142` | Lowercase token. The footer token is `Refs`. |","### Adoption","This guide is the rule for commits in this repository. Deviating from it","requires a justification — an ADR under `maintainers/adr/`, or an update to this","guide.","It applies from its adoption on, to every commit created after. Prior history is","not rewritten.","Enforcement stands today at pull-request review, which refuses a non-conforming","message and lets the author correct it before the merge. The repository will","lock the rule in turn: a `commit-msg` hook that refuses the message at write","time, doubled with a CI check, since a local hook can be bypassed. Merge","commits, generated by GitHub, are exempt.","### Credits","specification, published under [CC BY 3.0](https://creativecommons.org/licenses/by/3.0/).","## Pull request titles","The convention above governs each commit. A pull request needs a line of its","own, and it is not the same object: the commit is the unit of the change, the","**pull request the unit of the request** — the relation the doctrine already","draws between the commit and the issue. A pull request MAY therefore gather","several commits, of several types.","Its title is read in three places: the list of open pull requests, the","`Merge pull request #NN` commit GitHub writes when the branch lands (this","repository merges with a merge commit), and the draft of the release notes. It","earns the same care as a commit header. Unlike a commit, it is **not** linted;","it stands on the review, as the code does.","* The title MUST be in **English**, like everything else recorded here.","* It MUST name the **whole** pull request, not one of its commits. The per-commit"," types live in the commits, where the hook and CI check them; the title says"," what the branch delivers.","* Its shape follows from how many intentions the pull request carries:"," * **One intention** — the branch does a single kind of thing. The title MUST"," mirror the commit header it collapses to:"," `[()][!]: `, under the very rules of the section"," above — imperative present, lowercase after the colon, no trailing period. A"," one-commit pull request's title is that commit's header, verbatim."," * **Several intentions** — the branch carries a feature and the refactor that"," prepared it, or a fix and the test that pins it. The title MUST NOT borrow a"," single `type:` prefix: it would name one commit and hide the rest. It states"," the subject in plain words, as a title — an initial capital, no trailing"," period. A topical prefix (`Release supply chain: …`) is welcome; a"," Conventional-Commits type is not, unless it is honestly the only one.","* Keep the title within the **72 characters** a commit header targets, so the"," pull request list shows it whole.","* The issue reference lives in the **description**, never the title: `Closes #NN`"," when the pull request closes the issue, so GitHub closes it on merge; `Refs:"," #NN` otherwise. The title states the change, not where it was asked. A breaking"," change is signalled the same way it is on a commit — the `!` and the"," `BREAKING CHANGE:` note ride the commit, and the template's \"Breaking change\""," box repeats it — not the title.","| Title | Why it fits |","| `ci: add dependency review on pull requests` | One intention. The title is the commit header. |","| `feat(analyzers): add FCE016 for an undocumented error code` | One intention, scoped. The issue it closes lives in the description, not here. |","| `Adopt and enforce a Conventional Commits convention` | The guide, the hook, and the CI gate — several commits of several types. A plain title names them all. |","| `Release supply chain: build provenance + embedded SBOM` | Several intentions under one topic. A topical prefix carries it; no single `type:` would be honest. |","| Title | What is wrong |","| `feat: various improvements` | A type on a grab-bag. Either it is one intention — name it — or it is several, and `feat:` hides them. |","| `fix(core): Fixed the null dereference.` | The single-intention form, wearing the commit header's own faults: capital, past tense, trailing period. |","| `Add Outcome.Map (#142)` | The issue number belongs in the description's `Closes`/`Refs`, where GitHub reads it — not eating the title. |","| `Corrige le rendu des exemples` | Not English. |",[1012,1013,1014,1015,1016,1017,1018],{"_603":46,"_46":1028},{"_603":1020,"_161":456,"_624":1026,"_625":1027},{"_603":46,"_46":1025},{"_603":1020,"_161":456,"_624":1024,"_625":1022},{"_603":46,"_46":1023},{"_603":1020,"_161":456,"_624":1021,"_625":1022},{"_603":46,"_46":1019},"\n\nNo new changes were needed from me, so I did not create an additional commit or PR.\n\n### Checks run\n\n* ✅ `git status --short`\n* ✅ `git show --stat --oneline HEAD`\n* ✅ `nl -ba CONTRIBUTING.md | sed -n '70,104p'`","repo_file_citation",101,104,"\n* The text still protects shared branches by forbidding history rewrites once others may have based work on the branch. ",98,"\n* The commit-message remediation concern is addressed: the policy now permits rewriting history with `git rebase -i` / force-push while the branch is still the author’s own branch, including mid-review fixes for commit messages rejected by lint or review. ",76,80,"Yes — the fix looks OK for both review comments.\n\n### Review of requested fixes\n\n* The Dependabot concern is addressed: the policy now explicitly allows generated branch namespaces to keep their native layout, including `dependabot/nuget/Newtonsoft.Json-13.0.1`, while keeping `/` for branches cut by a person or agent by hand. ",{"_92":306,"_42":84,"_96":306,"_1031":1032,"_1033":1034,"_1035":1036,"_132":-5,"_119":1037},[83],"input_items",[1055,1056,1057,1058,1059],"external_post_config",{"_1038":1039,"_243":244,"_1040":286,"_255":-5,"_1041":237,"_1042":-5,"_1043":1044,"_1045":1046,"_1047":-5,"_1048":-5,"_1049":1050,"_1051":15,"_1052":59,"_1053":54,"_1054":15},"user_model_slug","slug",1783763862.830542,"installation_id",72237377,"pr_number","repo_id","review_trigger","comment_id",4944619797,"comment_type","issue","pr_write_guard_token","pending_pr_reaction_id","pending_comment_reaction_id",381319466,"comment_if_no_findings","comment_on_error","share_url","is_parallel_review",{"_92":381,"_96":306,"_375":1071},{"_293":294,"_295":296,"_1068":300,"_1069":129,"_302":-5,"_92":1070},{"_92":1060,"_295":1067,"_161":456,"_1062":1065,"_1063":-5},{"_92":1060,"_295":1064,"_161":456,"_1062":1065,"_1063":1066},{"_92":1060,"_295":1061,"_161":-5,"_1062":-5,"_1063":-5},"pull_request_comment","\n### 💡 Codex Review\n\nHere are some automated review suggestions for this pull request.\n\n**Reviewed commit:** `8cc6bfcab7`\n \n\n

ℹ️ About Codex in GitHub

","side","line","**_{_{![P2 Badge](https://img.shields.io/badge/P2-yellow?style=flat)}} Permit generated Dependabot branch names**\n\nWith the current `.github/dependabot.yml`, Dependabot branch names are not customized, and GitHub documents the default as `dependabot/PACKAGE_MANAGER/DEPENDENCY` (for example `dependabot/nuget/...`), not the two-component `/` shape required here. That makes routine NuGet and GitHub Actions Dependabot PRs non-conforming even though the new text says `dependabot/…` is part of the rule; either the policy needs to allow Dependabot's generated shape or the Dependabot config needs to be changed.\n\nUseful? React with 👍 / 👎.","RIGHT",73,"**_{_{![P2 Badge](https://img.shields.io/badge/P2-yellow?style=flat)}} Leave a path to fix rejected commit messages**\n\nIn PRs where commit-lint CI or the reviewer rejects a commit message, this new MUST NOT blocks the documented remediation path: the existing guide says review refuses non-conforming messages and lets the author correct them before merge, and the CI failure text explicitly tells authors to use `git rebase -i`. Because commit messages cannot be fixed by adding a follow-up commit, a branch that is already under review becomes impossible to bring into compliance unless this rule allows review-requested history fixes while the branch is not shared with collaborators.\n\nUseful? React with 👍 / 👎.","base_ref","head_ref","pull_request_info",[1072],{"_603":46,"_46":1073},"@codex Ok for the fix ?",[1259,1260],[],{"_608":609,"_46":610,"_611":612},{"_582":1250},{"_315":1249},{"_315":1109},{"_285":286,"_276":287,"_288":289,"_290":15,"_291":59,"_292":15,"_293":294,"_295":296,"_297":248,"_298":127,"_299":300,"_301":129,"_302":303,"_304":-5,"_305":-5,"_306":1108,"_308":309,"_310":309,"_311":312,"_313":32},{"_42":123,"_193":194,"_195":196,"_197":15,"_198":199,"_200":201,"_202":1097,"_204":-5,"_205":-5,"_206":1098,"_208":199,"_209":199,"_210":1099,"_212":1100,"_214":1101,"_216":-5,"_217":218,"_219":220,"_221":222,"_223":1102,"_55":-5,"_119":225,"_226":1103,"_228":1104,"_230":231,"_232":15,"_233":15},{"_133":1090},[1084,1085,1086,1087,1088,1089],{"_148":171,"_150":165,"_152":166,"_154":163,"_156":172,"_158":159,"_160":32,"_161":-5},{"_148":170,"_150":165,"_152":166,"_154":163,"_156":167,"_158":159,"_160":32,"_161":-5},{"_148":168,"_150":165,"_152":166,"_154":163,"_156":169,"_158":159,"_160":32,"_161":-5},{"_148":164,"_150":165,"_152":166,"_154":163,"_156":167,"_158":159,"_160":32,"_161":-5},{"_148":162,"_150":151,"_152":153,"_154":163,"_156":157,"_158":159,"_160":32,"_161":-5},{"_148":149,"_150":151,"_152":153,"_154":155,"_156":157,"_158":159,"_160":32,"_161":-5},[1091,1092,1093],{"_177":1096,"_179":190},{"_177":1095,"_179":186},{"_177":1094,"_179":180},{"_181":182,"_92":183,"_119":184},{"_181":187,"_92":183,"_119":188},{"_181":191,"_92":183,"_119":192},{"_278":279,"_280":-5,"_281":-5,"_282":-5,"_283":-5,"_284":-5},[237],{"_237":1105},{},{},[],{"_236":59},{"_234":59,"_235":199},{"_42":239,"_181":240,"_241":1106,"_243":244,"_245":246,"_247":248,"_249":250,"_251":252,"_253":254,"_255":64,"_256":15,"_257":-5,"_258":-5,"_259":-5,"_260":-5,"_261":-5,"_262":-5,"_160":263,"_264":-5,"_55":1107},{"_270":271,"_42":272,"_92":273,"_274":275,"_276":277},{"_266":15,"_267":15,"_268":15,"_269":15},{"_270":271,"_181":271,"_314":-5,"_274":275,"_42":272},[1110,1111,1112,1113,1114,1115,1116,1117,1118,1119,1120,1121,1122,1123,1124,1125,1126,1127,1128,1129,1130,1131,1132,1133,1134,1135,1136,1137,1138,1139,1140,1141,1142],{"_154":558,"_351":1245},{"_154":366,"_351":1242},{"_154":548,"_351":1239},{"_154":370,"_351":1234},{"_154":370,"_351":1230},{"_154":370,"_351":1226},{"_154":385,"_351":1221},{"_154":385,"_351":1217},{"_154":370,"_351":1214},{"_154":370,"_351":1212},{"_154":385,"_351":1208},{"_154":370,"_351":1206},{"_154":385,"_351":1202},{"_154":370,"_351":1199},{"_154":370,"_351":1197},{"_154":385,"_351":1193},{"_154":370,"_351":1191},{"_154":385,"_351":1187},{"_154":370,"_351":1184},{"_154":370,"_351":1182},{"_154":385,"_351":1178},{"_154":370,"_351":1176},{"_154":385,"_351":1172},{"_154":370,"_351":1169},{"_154":370,"_351":1167},{"_154":385,"_351":1163},{"_154":370,"_351":1161},{"_154":385,"_351":1157},{"_154":370,"_351":1154},{"_154":385,"_351":1152},{"_154":370,"_351":1148},{"_154":366,"_351":1146},{"_154":350,"_351":1143},{"_353":354,"_87":1144},{"_356":357,"_358":359,"_103":-5,"_42":360,"_361":1145,"_363":364,"_365":113},[],{"_365":1147,"_353":354},{"_92":369},{"_372":1149,"_353":354,"_374":360},{"_375":1150,"_377":-5,"_42":-5,"_378":-5,"_379":380,"_96":93,"_92":381},[1151],{"_46":383,"_92":384},{"_372":1153,"_353":354,"_374":360},{"_42":388,"_379":380,"_46":383,"_92":389},{"_372":1155,"_353":354,"_374":360},{"_375":-5,"_392":393,"_42":-5,"_378":-5,"_394":1156,"_92":396},[],{"_372":1158,"_353":354,"_374":360},{"_375":1159,"_42":400,"_394":1160,"_92":396},[],[],{"_372":1162,"_353":354,"_374":360},{"_404":405,"_42":-5,"_378":-5,"_406":407,"_92":408},{"_372":1164,"_353":354,"_374":360},{"_411":412,"_413":414,"_415":1165,"_417":418,"_358":32,"_419":32,"_42":405,"_420":421,"_422":423,"_365":113,"_92":424},[1166],{"_413":426,"_92":427},{"_372":1168,"_353":354,"_374":360},{"_430":431,"_404":405,"_42":-5,"_378":-5,"_181":432,"_433":-5,"_92":434},{"_372":1170,"_353":354,"_374":360},{"_375":-5,"_392":437,"_42":-5,"_378":-5,"_394":1171,"_92":396},[],{"_372":1173,"_353":354,"_374":360},{"_375":1174,"_42":442,"_394":1175,"_92":396},[],[],{"_372":1177,"_353":354,"_374":360},{"_404":446,"_42":-5,"_378":-5,"_406":447,"_92":408},{"_372":1179,"_353":354,"_374":360},{"_411":450,"_413":451,"_415":1180,"_417":418,"_358":32,"_419":32,"_42":446,"_420":453,"_422":423,"_365":113,"_92":424},[1181],{"_413":455,"_181":456,"_161":457,"_92":458},{"_372":1183,"_353":354,"_374":360},{"_430":461,"_404":446,"_42":-5,"_378":-5,"_181":432,"_433":-5,"_92":434},{"_372":1185,"_353":354,"_374":360},{"_375":-5,"_392":464,"_42":-5,"_378":-5,"_394":1186,"_92":396},[],{"_372":1188,"_353":354,"_374":360},{"_375":1189,"_42":469,"_394":1190,"_92":396},[],[],{"_372":1192,"_353":354,"_374":360},{"_404":473,"_42":-5,"_378":-5,"_406":474,"_92":408},{"_372":1194,"_353":354,"_374":360},{"_411":477,"_413":478,"_415":1195,"_417":418,"_358":32,"_419":32,"_42":473,"_420":480,"_422":423,"_365":113,"_92":424},[1196],{"_413":482,"_92":427},{"_372":1198,"_353":354,"_374":360},{"_430":485,"_404":473,"_42":-5,"_378":-5,"_181":432,"_433":-5,"_92":434},{"_372":1200,"_353":354,"_374":360},{"_375":-5,"_392":488,"_42":-5,"_378":-5,"_394":1201,"_92":396},[],{"_372":1203,"_353":354,"_374":360},{"_375":1204,"_42":493,"_394":1205,"_92":396},[],[],{"_372":1207,"_353":354,"_374":360},{"_404":497,"_42":-5,"_378":-5,"_406":498,"_92":408},{"_372":1209,"_353":354,"_374":360},{"_411":501,"_413":502,"_415":1210,"_417":418,"_358":32,"_419":309,"_42":497,"_420":504,"_422":423,"_365":505,"_92":424},[1211],{"_413":507,"_92":427},{"_372":1213,"_353":354,"_374":360},{"_430":510,"_404":497,"_42":-5,"_378":-5,"_181":432,"_433":-5,"_92":434},{"_372":1215,"_353":354,"_374":360},{"_375":-5,"_392":513,"_42":-5,"_378":-5,"_394":1216,"_92":396},[],{"_372":1218,"_353":354,"_374":360},{"_375":1219,"_42":518,"_394":1220,"_92":396},[],[],{"_372":1222,"_353":354,"_374":360},{"_522":-5,"_375":1223,"_42":524,"_92":525},[1224],{"_46":527,"_528":1225,"_92":46},[],{"_372":1227,"_353":354,"_374":360},{"_375":1228,"_377":-5,"_42":-5,"_378":-5,"_379":-5,"_96":306,"_92":381},[1229],{"_46":527,"_92":534},{"_372":1231,"_353":354,"_374":360},{"_375":1232,"_377":-5,"_42":-5,"_378":-5,"_379":-5,"_96":306,"_92":381},[1233],{"_46":539,"_92":534},{"_372":1235,"_353":354,"_374":360},{"_375":1236,"_377":-5,"_42":-5,"_378":-5,"_379":-5,"_96":543,"_92":381},[1237,1238],{"_46":547,"_92":534},{"_46":546,"_92":534},{"_353":354,"_87":1240},{"_103":-5,"_42":360,"_361":1241,"_363":364,"_365":552},[],{"_365":1243,"_353":354},{"_555":1244,"_92":557},[],{"_560":1246},{"_562":-5,"_563":-5,"_564":565,"_566":364,"_417":418,"_567":15,"_568":-5,"_569":-5,"_42":354,"_570":571,"_181":-5,"_572":-5,"_161":573,"_574":199,"_575":-5,"_422":576,"_365":1247,"_578":-5,"_77":1248,"_580":364},{"_92":369},[],[],[1251],{"_42":585,"_586":1252,"_588":589,"_590":589,"_375":1253,"_365":592,"_377":-5,"_593":32,"_594":1254,"_596":597,"_598":-5},{"_96":306,"_181":-5,"_594":1258},{"_603":46,"_604":1257},{"_599":1255},{"_601":1256},[],[606],{},{"_92":381,"_96":93,"_375":1266},{"_92":615,"_616":1261},[1262],{"_161":456,"_619":-5,"_620":1263,"_622":-5},[1264],{"_624":309,"_625":626,"_375":1265,"_628":59},[629,199,630,631,632,199,633,199,634,635,636,637,199,638,639,199,640,199,641,642,199,643,644,643,199,645,646,647,648,199,649,650,199,651,199,652,199,653,654,655,656,657,658,199,643,659,660,661,662,643,199,663,664,665,199,666,199,667,668,669,670,671,672,673,199,674,675,676,674,677,678,679,680,681,682,683,684,685,686,687,688,689,690,691,692,693,694,695,696,697,698,699,700,701,702,703,704,705,706,707,708,709,710,711,712,713,199,674,675,714,715,674,199,716,717,199,718,199,719,720,721,199,722,723,724,725,726,727,199,728,729,730,199,731,732,733,199,734,735,736,199,737,738,739,199,740,199,741,742,743,744,745,746,199,747,199,748,742,749,750,751,752,753,754,755,756,757,199,758,199,759,760,761,762,199,652,199,763,764,199,643,765,766,767,643,199,768,199,643,769,770,771,643,199,772,773,774,775,199,666,199,776,777,778,779,199,780,199,643,781,199,782,199,783,643,199,784,785,786,787,788,789,790,199,791,199,792,793,199,794,795,796,797,798,799,800,801,802,803,804,805,806,199,807,808,199,809,199,810,199,811,742,812,813,814,815,816,199,817,818,819,199,820,821,822,823,824,825,826,199,827,828,829,830,199,643,831,643,199,832,199,833,834,835,836,837,838,839,840,841,199,842,199,843,844,845,199,846,847,848,199,849,199,850,851,852,853,199,854,855,856,857,858,199,859,860,861,862,863,864,199,865,866,867,868,199,869,199,870,871,199,643,872,199,873,874,875,643,199,876,877,878,199,879,880,881,882,199,883,199,884,885,199,643,886,199,887,643,199,888,889,890,891,199,718,199,892,893,894,895,199,896,897,898,899,900,199,901,902,903,904,199,905,906,907,199,908,909,199,910,911,912,199,913,914,915,199,916,917,918,919,920,199,921,922,923,924,199,740,199,925,199,643,926,199,927,643,199,928,199,643,929,199,930,931,932,199,933,643,199,934,935,199,643,936,643,199,937,199,643,872,199,873,874,875,199,938,643,199,747,199,939,199,940,742,941,942,943,944,945,946,947,948,949,950,951,952,199,953,199,954,955,956,199,957,958,199,959,960,961,962,963,199,964,199,759,965,199,966,199,967,968,969,970,971,199,972,973,974,975,976,199,666,199,977,978,979,980,981,982,983,984,985,986,987,988,989,990,991,992,993,994,995,996,997,998,999,1000,199,740,199,1001,742,1002,1003,1004,1005,199,747,199,1006,742,1007,1008,1009,1010],[1267,1268,1269,1270,1271,1272,1273],{"_603":46,"_46":1028},{"_603":1020,"_161":456,"_624":1026,"_625":1027},{"_603":46,"_46":1025},{"_603":1020,"_161":456,"_624":1024,"_625":1022},{"_603":46,"_46":1023},{"_603":1020,"_161":456,"_624":1021,"_625":1022},{"_603":46,"_46":1019},[1276,1277,1278,1279,1280],{"_1038":1039,"_243":244,"_1040":286,"_255":-5,"_1041":237,"_1042":-5,"_1043":1044,"_1045":1046,"_1047":-5,"_1048":-5,"_1049":1050,"_1051":15,"_1052":59,"_1053":54,"_1054":15},{"_92":381,"_96":306,"_375":1281},{"_293":294,"_295":296,"_1068":300,"_1069":129,"_302":-5,"_92":1070},{"_92":1060,"_295":1067,"_161":456,"_1062":1065,"_1063":-5},{"_92":1060,"_295":1064,"_161":456,"_1062":1065,"_1063":1066},{"_92":1060,"_295":1061,"_161":-5,"_1062":-5,"_1063":-5},[1282],{"_603":46,"_46":1073},"task_e_6a521395d13083209b62be03f7e57fd2",1783763861.817253,"GitHub Mention: docs: add a branch-management policy","has_generated_title","has_unread_turn","denormalized_metadata",{"_1314":32,"_1315":1316,"_1317":-5,"_1318":-5,"_1309":1319,"_1320":-5,"_1321":-5,"_1322":-5,"_1323":-5},"task_status_display",{"_1302":1303,"_1304":196,"_130":127,"_1305":-5,"_1306":1307},"creator_workspace","external_pull_requests",[1295],{"_42":1296,"_1297":83,"_1298":1299,"_1300":129},"github-1046229690-97","assistant_turn_id","pull_request",{"_285":286,"_276":287,"_288":289,"_290":15,"_291":59,"_292":15,"_293":294,"_295":296,"_297":248,"_298":127,"_299":300,"_301":129,"_302":303,"_304":-5,"_305":-5,"_306":1301,"_308":309,"_310":309,"_311":312,"_313":32},"codex_updated_sha",{"_270":271,"_181":271,"_314":-5,"_274":275,"_42":272},"latest_turn_status_display",{"_1308":83,"_112":113,"_1309":-5,"_94":1310,"_1311":-5,"_1312":-5,"_1313":1307,"_114":-5},"environment_label","fix_task_id","initial_intent","pr","turn_id","diff_stats",[],"image_asset_pointer_citations","issue_counts","intent","num_follow_ups","repo_name_map",{},"last_user_turn_created_at","last_turn_status",{"_1324":-5,"_1325":-5,"_1326":-5},"last_pushed_turn_id","start_branch_name","pushed_branch_name","cached_pull_request_data","files_modified","lines_added","lines_removed","dd",{"_1338":1339,"_1340":1341},"disablePrefetch","shouldPrefetchAccount","shouldPrefetchUser","shouldPrefetchSystemHints","promoteCss","disableSSR","stripModulepreloadImports","statsigGateEvaluationsPromise",["P",1337],"traceId","14559054564991952716","traceTime",1783774730031]