Skip to content

docs: add annotated refund violation trace - #49

Merged
CodewithJha merged 1 commit into
CodewithJha:mainfrom
Nomee-123:docs/annotated-refund-trace
Sep 10, 2026
Merged

docs: add annotated refund violation trace#49
CodewithJha merged 1 commit into
CodewithJha:mainfrom
Nomee-123:docs/annotated-refund-trace

Conversation

@Nomee-123

Copy link
Copy Markdown
Contributor

Summary

  • Add an annotated companion for m4_refund_limit_violation.json
  • Explain how the refund_limit policy maps to the issue_refund tool call
  • Document the evidence fields that show why the rule is violated

Closes #40

Type of change

Checklist

  • I read CONTRIBUTING.md (where to ask, how reviews work, how to run tests)
  • Linked related issue (Fixes #N) or Discussion when applicable
  • Core stays free of framework SDK imports (adapters own glue)
  • Tests added or updated when behavior changes (uv run pytest tests/unit -q)
  • Docs / README claims match reality (no PyPI / multi-adapter claims unless shipped)
  • One concern per PR when possible
  • Authorized testing only — no open-internet attack-proxy behavior

Test plan

docs-only / N/A

Signed-off-by: Nomee Singh <singhnomee@gmail.com>
@vercel

vercel Bot commented Sep 10, 2026

Copy link
Copy Markdown

@Nomee-123 is attempting to deploy a commit to the priyanshu's projects Team on Vercel.

A member of the Team first needs to authorize it.

@CodewithJha CodewithJha left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM — S-sized docs-only companion for #40: m4_refund_limit_violation.annotated.md maps refund_limitissue_refund (demo-tc-1, amount 210, approved false) with clear evidence notes. No Core changes; unit CI green on 3.11/3.12. Ignoring Vercel auth.

@CodewithJha

Copy link
Copy Markdown
Owner

Thanks @Nomee-123 — this hits #40 cleanly: annotated sibling for the refund-limit violation trace, links the refund_limit rule, and stays static/docs-only (no campaign or real refund). Unit CI green on 3.11/3.12; ignoring the Vercel auth failure (unrelated to this PR). Merging now.

@CodewithJha
CodewithJha merged commit 0b4673d into CodewithJha:main Sep 10, 2026
5 of 6 checks passed
@CodewithJha

Copy link
Copy Markdown
Owner

Merged — thank you @Nomee-123. Exactly the kind of good-first-issue docs companion Mutiny needs: clear refund_limit evidence mapping, static/authorized-testing only, no Core churn.

If you want another similar bite later, feel free to browse open good first issue labels — comment on the issue first so nobody doubles up.

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.

examples: annotated violation trace with evidence notes

2 participants