Skip to content

docs: clarify parity registry references - #208

Merged
Paul-Kyle merged 1 commit into
phasespace-labs:mainfrom
Utkarsh3725:Utkarsh3725-patch-1
Sep 13, 2026
Merged

Paul-Kyle merged 1 commit into
phasespace-labs:mainfrom
Utkarsh3725:Utkarsh3725-patch-1

Conversation

@Utkarsh3725

@Utkarsh3725 Utkarsh3725 commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

Fixes #173

@Paul-Kyle

Copy link
Copy Markdown
Member

Thank you for bringing this upstream — and both cleanups are exactly right: the period is back and the blank line before ## Surface sugar survived.

One citation still remains. Line 118 of docs/PARITY.md currently reads

Stored at `palinode/core/parity.py:PROMPT_TASKS`. The ADR-010 parity pass fixed the duplicate-`enum` bug at `palinode/mcp.py:624-625`; the canonical list now lives in `parity.py`.

and it should read

Stored at `palinode/core/parity.py:PROMPT_TASKS`. The ADR-010 parity pass fixed the duplicate-`enum` bug; the canonical list now lives in `parity.py`.

The line it cites now holds MCP_SEARCH_LIMIT_MAX = 50, so there is nothing to look at there — and unlike the other two there's no symbol to repoint to, which is why it's a deletion rather than a repoint.

Push that onto the same branch and I'll review and merge once checks are green. Could you also add Fixes #173 to the PR description? Without it the issue stays open after the merge, and the work in this PR is what closes it.

If you'd rather stop here, say so and I'll merge this as it stands and finish the last line myself — the credit for the issue stays yours either way.

Updated references to memory categories and entity-ref prefixes for clarity and accuracy.
@Paul-Kyle
Paul-Kyle merged commit cfbe705 into phasespace-labs:main Sep 13, 2026
13 checks passed
@Paul-Kyle

Copy link
Copy Markdown
Member

Merged — thank you, and thanks for the patience; this took three rounds to land through no fault of the work.

The third edit is the one worth naming. There was no symbol to repoint to, so the only correct move was to drop the dead pointer and keep the history sentence — and that is what you did. The other two go to memory_write.py, where those names are defined rather than imported, which is what makes the citation stable.

Every citation you wrote names a file and a symbol; none names a line number. That was the whole of this issue.

I'll add you to AUTHORS.md on our side — it reaches this repository with the next release sync rather than with this merge. If you'd like another issue, say so and I'll point you at one.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: three source references in PARITY.md point at the wrong lines

2 participants