Skip to content

Console code still cites console.md as saying the console is view-only #22

Description

@Pijukatel

src/console/server.ts justifies using listRequests rather than getHead/peekHead with "the console is documented as view-only (console.md)". requirements/console.md has said the opposite since before this comment — it records the console's writes and states it is no longer strictly view-only.

test/integration/console.test.ts has the same staleness in a test name ("view-only local dev console").

The comment's actual argument is sound and unaffected: viewing a page must not mutate queue state. Only the citation is wrong.

Pre-existing — both lines are byte-identical at the commit before the upstream-fallback work (#20), where this was found and reported each review round.

Activity

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

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions