Skip to content

feat(signals): add contribution policy snapshot API#369

Open
MkDev11 wants to merge 8 commits into
JSONbored:mainfrom
MkDev11:feat/repo-owner-policy-snapshots
Open

feat(signals): add contribution policy snapshot API#369
MkDev11 wants to merge 8 commits into
JSONbored:mainfrom
MkDev11:feat/repo-owner-policy-snapshots

Conversation

@MkDev11
Copy link
Copy Markdown
Contributor

@MkDev11 MkDev11 commented Jun 4, 2026

Summary

  • Add protected internal read/write API endpoints for repo-owner contribution policy snapshots.
  • Persist API-derived policy snapshots through the existing focus-manifest signal snapshot path and keep explicit api_record snapshots durable instead of expiring them like repo-file fetch caches.
  • Return normalized focus-manifest data plus the compiled policy partition so internal callers can distinguish public-safe guidance from authenticated maintainer context.
  • Closes feat(signals): add repo-owner contribution policy snapshots #111.

Scope

  • This PR is focused and does not mix unrelated backend, UI, MCP, docs, dependency, and deploy changes.
  • This follows CONTRIBUTING.md and does not reintroduce GitHub Pages, VitePress, site/, or CNAME.
  • I linked an issue, or this is small enough that the summary explains why an issue is not needed.

Validation

  • git diff --check
  • npm run actionlint
  • npm run typecheck
  • npm run test:coverage locally; global coverage stays at or above 97% for lines, statements, functions, and branches (aim for 98%+ branch coverage locally so CI variance does not fail near the threshold)
  • npm run test:workers
  • npm run build:mcp
  • npm run test:mcp-pack
  • npm run ui:openapi:check
  • npm run ui:lint
  • npm run ui:typecheck
  • npm run ui:build
  • npm audit --audit-level=moderate
  • New or changed behavior has unit/integration tests for new branches, fallback paths, and sanitizer boundaries

If any required check was skipped, explain why:

  • None. Full npm run test:ci passed locally. Coverage summary: statements 99.05%, branches 97.06%, functions 98.36%, lines 99.65%.

Safety

  • No secrets, wallet details, hotkeys, coldkeys, user PATs, private keys, raw trust scores, private rankings, or private maintainer evidence are exposed.
  • Public GitHub text stays sanitized, low-noise, and does not imply compensation guarantees or optimization tactics.
  • Auth, cookie, CORS, GitHub App, Cloudflare, or session changes include negative-path tests.
  • API/OpenAPI/MCP behavior is updated and tested where needed.
  • UI changes use live API data or real empty/error/loading states, not production mock/demo fallbacks.
  • Visible UI changes include screenshots or a short recording.
  • Public docs/changelogs are updated where needed; changelogs are only edited for release-prep PRs.

Notes

  • No visible UI changes.
  • Internal policy snapshot responses include authenticated maintainer context; public readiness output is tested to avoid leaking the private note text or forbidden public-report terms.

@MkDev11 MkDev11 requested a review from JSONbored as a code owner June 4, 2026 06:27
@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jun 4, 2026
@github-actions github-actions Bot added the feature New feature or request label Jun 4, 2026
Copy link
Copy Markdown
Owner

@JSONbored JSONbored left a comment

Choose a reason for hiding this comment

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

@MkDev11 please fix merge conflicts.

@MkDev11
Copy link
Copy Markdown
Contributor Author

MkDev11 commented Jun 4, 2026

@MkDev11 please fix merge conflicts.

please check the update

@gittensory
Copy link
Copy Markdown

gittensory Bot commented Jun 5, 2026

Important

Gittensory found maintainer review notes

Scoped related-work signals were found for this PR. They are advisory unless the gate reports a blocker.

Signal Status Action
Linked issue #111 Ready for issue-scoped review.
Duplicate risk 9+ scoped related-work signals Advisory review only.
Review burden High Estimated from public PR metadata.
Repo queue Critical Repo-wide context only; this does not block the gate.
Contributor context Confirmed Gittensor contributor Official public API confirmed this GitHub user.
Gate result Passing No configured blocker found.
Review context
Maintainer notes
  • Possible duplicate or overlapping work: 90 related open work cluster(s) were detected.
Contributor next steps
  • Review the scoped related-work signal before deep review; it is not a merge blocker by itself.
  • Check active issues and PRs before submitting.

Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers. Learn more about Gittensor contribution workflows.

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

Labels

feature New feature or request gittensory:reviewed Gittensor contributor context size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

feat(signals): add repo-owner contribution policy snapshots

2 participants