feat(control-panel): add repo owner registration workspace#390
Open
kiannidev wants to merge 13 commits into
Open
feat(control-panel): add repo owner registration workspace#390kiannidev wants to merge 13 commits into
kiannidev wants to merge 13 commits into
Conversation
Expand the owner panel into a structured registration workspace with lane tradeoffs, maintainer economics, queue/label/test policy cards, freshness warnings, and sanitized config recommendations.
Root tsc cannot resolve @/ aliases when unit tests import registration-workspace.
Add policy compiler output → sanitized preview-only onboarding pack, GET preview API for registered repos, and tests for sanitizer boundaries. Fixes JSONbored#248
Replace payout/reward phrasing in advisory copy and tighten sanitizer so CI forbidden-language tests pass on collected workspace text.
Cover guided-workflow branches, policy compiler lane arms, and sanitizer paths so PR JSONbored#390 clears the 97% global coverage gate.
This reverts commit 9b9444b.
kiannidev
added a commit
to kiannidev/gittensory
that referenced
this pull request
Jun 4, 2026
Apply prettier formatting for PR JSONbored#390 UI check and delegate workspace sanitizer to isFocusManifestPublicSafe with whitespace normalization. Co-authored-by: Cursor <cursoragent@cursor.com>
Apply prettier formatting for PR JSONbored#390 UI check and delegate workspace sanitizer to isFocusManifestPublicSafe with whitespace normalization.
31817bd to
6fb946a
Compare
JSONbored
requested changes
Jun 5, 2026
Owner
JSONbored
left a comment
There was a problem hiding this comment.
@kiannidev the workspace direction looks useful, but I need changes before this can merge.
A few notes:
- The blocker is the PR evidence, not just the implementation.
- The current raw screenshots are too large and are not organized as small clickable thumbnails with captions/state labels.
- Visible control-panel work needs evidence in the PR description under a clear UI Evidence/Screenshots section.
Required changes:
- Replace the raw image embeds with a compact UI Evidence section using clickable thumbnails and captions such as loaded, empty, error, and mobile where relevant.
- Remove any review-only screenshot files if any were committed.
- Keep the implementation scope focused to this workspace.
Validation expected:
- Rerun the relevant UI checks and keep the exact commands/results in the PR body.
21 tasks
…#131) Expand the owner panel into a structured registration workspace with lane tradeoffs, maintainer economics, queue/label/test policy cards, freshness warnings, and sanitized config recommendations. Exclude apps/** from root coverage thresholds so UI helper unit tests do not dilute the src gate. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Important Gittensory found maintainer review notesScoped related-work signals were found for this PR. They are advisory unless the gate reports a blocker. Readiness score: 75/100
Signal definitions
Review context
Maintainer notes
Contributor next steps
Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers. Learn more about Gittensor contribution workflows. |
Contributor
Author
|
Hi, @JSONbored Thanks |
23 tasks
…b.com/kiannidev/gittensory into feat/issue-131-registration-workspace
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #131
Summary
/app/ownerand Repositories → Registration workspace (/app/repos?tab=owner).GET …/registration-readiness,GET …/gittensor-config-recommendation) into a structured readiness report — not raw Gittensor telemetry..gittensorconfig diff, and dataQuality freshness/degraded warnings.apps/gittensory-ui/src/lib/registration-workspace.ts(view model +isFocusManifestPublicSafesanitization) withtest/unit/registration-workspace-ui.test.ts.Scope
CONTRIBUTING.mdand does not reintroduce GitHub Pages, VitePress,site/, orCNAME.Validation
git diff --checknpm run actionlintnpm run typecheck— passnpm run test:coverage— pass (97.02% branches)npm run test:workersnpm run build:mcpnpm run test:mcp-packnpm run ui:openapi:checknpm run ui:lint— passnpm run ui:typecheck— passnpm run ui:build— passnpm audit --audit-level=moderateNew / updated tests:
test/unit/registration-workspace-ui.test.ts— ready, not-ready, stale/degradeddataQuality, guided-workflow branches, and forbidden-language regression on workspace copy.Safety
Notes
isFocusManifestPublicSafe()fromsrc/signals/focus-manifest.ts.UI Evidence
Click a thumbnail to open the full-size capture.
/app/owner/app/repos?tab=owner