Skip to content

Add dotnet-service policy to the registry - #130

Open
ShubhaKC123 wants to merge 1 commit into
Signetry:mainfrom
ShubhaKC123:add-dotnet-service-policy
Open

ShubhaKC123 wants to merge 1 commit into
Signetry:mainfrom
ShubhaKC123:add-dotnet-service-policy

Conversation

@ShubhaKC123

Copy link
Copy Markdown

What & why

Adds a dotnet-service registry policy for conventional .NET src/ and tests/ repositories. It defines the permitted paths, blocks sensitive build and deployment configuration, documents the reasons for those restrictions, and requires dotnet build --warnaserror and dotnet test --no-build.

Also adds the policy to the registry guide. Addresses #118.

Checklist

  • Tests added/updated and uv run pytest passes — not run; uv is unavailable in this environment.
  • uv run ruff check . passes — not run; uv is unavailable in this environment.
  • No model/network calls added to the deterministic pipeline (agent/LLM work stays behind the Executor protocol or the optional classifier hook)
  • Core invariants preserved (auto_merge always false; authority earned from evidence; verifier never fabricates a pass; receipts verify against a pinned key)
  • Output/docs stay honest (enforcement tiers, model provenance, detection scope not overstated)
  • If security-relevant, I've noted the impact and threat-model implications

Notes for reviewers

The registry test suite validates each policy’s declared blocked and allowed paths against the real contract evaluator. git diff --check passed. The focused policy test could not be run because Python and uv are unavailable in the environment.

@github-actions

github-actions Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

All contributors have signed the CLA. ✅
Posted by the CLA Assistant Lite bot.

@ShubhaKC123

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

github-actions Bot added a commit that referenced this pull request Oct 27, 2026
@github-actions

Copy link
Copy Markdown

Signetry Reviewer — 🟡 Needs human review

A human should decide — the required check is pending.

Deterministic gates (the authority)

Gate Status
Required status check ⏳ pending
Secret scan ✅ clean
CI permission / OIDC ✅ no forbidden change
Dependency skew ✅ ok
All green ❌

Findings

No issues found by the deterministic scanners.

Merge

A human should review and merge.

This review is advisory. It never merges on its own judgement — the deterministic gates + a human are the authority. Findings can have false negatives; a green bot verdict is not a guarantee.

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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant