Skip to content

ProjectStateStore.read_maintainers has no production reader #608

Open

Description

@mlieberman85

After #588 removes write_project/write_maintainers, ProjectStateStore.read_maintainers is the remaining protocol member with no production caller (wiring audit CORE-7). Its two implementations also disagree: the filesystem store derives entries from project.maintainer_entries, while the in-memory store returns [].

Either give it a reader (the maintainers context key is the obvious one), or remove it from the protocol and both stores, with a BREAKING entry.

Drafted with Claude Code; reviewed and posted by me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    module-coredarnit core frameworkneeds triageAwaiting maintainer review

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions