Skip to content

docs(parmana-server): fix Parmana link and correct RFC 8785 wording - #415

Open
pavancharak wants to merge 3 commits into
agentrust-io:mainfrom
pavancharak:patch-3
Open

pavancharak wants to merge 3 commits into
agentrust-io:mainfrom
pavancharak:patch-3

Conversation

@pavancharak

@pavancharak pavancharak commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

Two fixes to integrations/parmana-server/README.md:

  • The Parmana link on line 3 now points to pavancharak/parmana-core. It previously pointed to pavancharak/parmana.
  • The canonical-form sentence said Parmana signs "not RFC 8785". For plain JSON values Parmana's serializer matches RFC 8785, and the parmana-core tests in pavancharak/parmana-core#317 pin that. The two non-JSON differences (undefined values dropped, Date values as ISO strings) are now stated.

This is documentation only. It does not change the verified fixture path or the tier.

Supersedes #414, which changes the same line 3 but with a broken link.

docs(parmana-server): fix Parmana link and correct RFC 8785 wording
@github-actions

github-actions Bot commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

🔴 Contributor Check: HIGH

Check Result
Profile HIGH
Credential LOW
Overall HIGH

Automated check by AgenTrust Contributor Check.

@github-actions github-actions Bot added the needs-review:HIGH Contributor check flagged HIGH risk label Nov 4, 2026
@pavancharak pavancharak mentioned this pull request Nov 4, 2026
@pavancharak

Copy link
Copy Markdown
Contributor Author

docs(parmana-server): fix typo on line 8

@pavancharak

Copy link
Copy Markdown
Contributor Author

@carloshvp @imran-siddique when you have a moment, could you review this? It's a README-only change: it fixes the Parmana link, replaces the "not RFC 8785" wording with a checked statement, and fixes a typo on line 7. The conformance tests are in pavancharak/parmana-core#317.
)_

@drk-ram drk-ram left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed commit c7626d6. The README-only update fixes the repository target and sentence typo, and accurately scopes canonicalization compatibility to plain JSON while documenting the two non-JSON input behaviors. The linked conformance coverage is merged and passing. Approved.

This branch has not been deployed

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

Labels

needs-review:HIGH Contributor check flagged HIGH risk

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants