[MichaelMure]

Confidence Score: 4/5

Do not merge yet: broken commondir still blocks otherwise valid protected and file-only loads.

Summary

Implements Git configuration reads and edits, including scoped loads, conditional includes, typed getters, and locked file updates.

• Adds EnvError so callers can identify malformed environment variables.
• Clarifies API behavior and adds tests for error messages.
• Replaces the reverse global-file loop with slices.Backward without changing file selection.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A["Discover or caller-supplied Env"] --> B["Load scopes or files"]
  B --> C["Follow matching includes"]
  C --> D["Immutable Config"]
  E["Parse bytes"] --> F["Editable File"]
  F --> D
  F --> G["Edit and check entries"]
  H["UpdateFile or UpdateScope"] --> I["Lock and read target"]
  I --> G
  G --> J["Write lock and rename"]

Loading

_{Reviews (9) · Last reviewed commit: "improve an error case" · Reviewed by Greptile}

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.