Skip to content

feat: add a Parmana trust-record verifier integration - #4427

Open
Pavan Dev Singh Charak (pavancharak) wants to merge 1 commit into
microsoft:mainfrom
pavancharak:parmana-trust-verify
Open

Pavan Dev Singh Charak (pavancharak) wants to merge 1 commit into
microsoft:mainfrom
pavancharak:parmana-trust-verify

Conversation

@pavancharak

Copy link
Copy Markdown

Related Issue

None. The Problem & Solution section below stands in for a linked issue.

Problem & Solution

Problem: AgentMesh workflows have no offline check for Parmana execution trust records. A workflow that acts on a record cannot tell whether a pinned key signed it, unless it re-implements the verifier.
Solution: a community extension, agentmesh-parmana-trust-verify. verify_record(record, trust_document) uses the offline parmana-verifier package. It returns a verdict that denies on any failure: a malformed trust set, a non-object record, or an exception during verification.

Impact on Your Work

Lets an AgentMesh workflow gate its own actions on a signed Parmana record, with no network call and no trust in Parmana's servers.

Timeline

None.

Alternatives Considered

Re-implementing Ed25519 verification inside AgentMesh. Rejected: it would duplicate the verifier's canonicalization, which must match exactly.

Type of Change

  • New feature (non-breaking change that adds functionality)

Package(s) Affected

  • agentmesh-integrations (framework integrations)

Testing

Unit Testing

7 tests, all passing. They cover a valid record under a pinned key, a tampered transaction, a record signed by an unpinned key, a record without a signature, a malformed trust set, a non-object record, and that the input record is not mutated. The tests build real signed records with the verifier's own canonicalization.

Manual Testing

python examples/basic_usage.py prints allowed: True, then allowed: False after tampering. ruff check and ruff format pass with the pinned 0.12.4.

Checklist

  • I have completed "Problem & Solution", "Impact on Your Work", and "Alternatives Considered" (no linked issue)
  • My code follows the project style guidelines (ruff check)
  • I have added tests that prove the feature works
  • All new and existing tests pass
  • I have updated documentation as needed (README)
  • I have signed the Microsoft CLA (to be done by the PR author)

Attribution & Prior Art

  • This contribution does not contain code copied or derived from other projects without attribution
  • Related project: Parmana, pavancharak/parmana-core. The verifier is parmana-verifier on PyPI (Ed25519).

AI Assistance

An AI coding assistant drafted the extension, tests, and README. The PR author reviewed every change and ran the tests and lint checks.

IP, Patents, and Licensing

  • This contribution does not implement patent-pending or patent-encumbered techniques
  • This contribution does not require an NDA or licensing agreement to understand or use
  • Any AI tools used have terms compatible with the MIT License

A community extension that checks a Parmana execution trust record against
keys the caller pins, using the offline parmana-verifier package. It denies
on any failure, including a malformed trust set or a verifier exception.
Tests build real signed records with the verifier's canonicalization.

Signed-off-by: pavancharak <charak1987@gmail.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@github-actions github-actions Bot added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file tests size/L Large PR (< 500 lines) labels Oct 26, 2026
@github-actions

Copy link
Copy Markdown

PR Review Summary

Check Status Details
🔍 Code Review ⚠️ Missing No current-run comment
🛡️ Security Scan ⚠️ Missing No current-run comment
🔄 Breaking Changes ⚠️ Missing No current-run comment
📝 Docs Sync ⚠️ Missing No current-run comment
🧪 Test Coverage ⚠️ Missing No current-run comment

Verdict: ⚠️ AI review incomplete; ready for human review

AI review comments are untrusted advisory output. The summary reports workflow-generated completion status only, not model-authored pass/fail claims.

@github-actions

Copy link
Copy Markdown

🟡 Contributor Check: MEDIUM

Check Result
Profile MEDIUM
Credential LOW
Overall MEDIUM

Automated check by AGT Contributor Check.

@github-actions github-actions Bot added the needs-review:MEDIUM Contributor check flagged MEDIUM risk label Oct 26, 2026
@pavancharak

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

San-Hsien (SanHsien) added a commit to SanHsien/agent-governance-toolkit that referenced this pull request Oct 26, 2026
…oft#4426, advance watermarks

- 6dc2093 (microsoft#4418): code and tests already in the fork; sync the reward
  computation spec so it matches the applied-penalty behaviour
- 46b370e (microsoft#4259 --sync-registry): not adopted; generate_workflows.py
  --check is already red in the fork (hand-added repository gates) and the
  new symlink tests raise WinError 1314 on unprivileged Windows
- PR microsoft#4427 (Parmana verifier integration): not adopted; head
  bc596b0 scanned for hidden content, clean
- issue microsoft#4426 (ApprovalResolution signature RFC): WATCH
- baseline: commit 46b370e, PR 4427, issue 4426

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

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

dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation needs-review:MEDIUM Contributor check flagged MEDIUM risk size/L Large PR (< 500 lines) tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant