Repository navigation
(janitor/comments): remove narration, banners, and commented-out code from admin console - #8900
Merged
Merged
Conversation
… from admin console Delete 272 comment lines across the admin console area (apps/web/src/app/admin): section banner separators, step-narration comments, state-group labels, and commented-out code. Comments explaining business rules, security rationale, and non-obvious constraints are retained.
Contributor
Author
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Executive SummaryThe PR removes 272 comment lines across 48 files under Assumptions: Verification Notes
Files Reviewed (48 files)
Reviewed by deepseek-flash-latest · Input: 61.9K · Output: 9.1K · Cached: 507.6K Review guidance: REVIEW.md from base branch |
Contributor
Author
|
Meta Janitor: Safety review passed; requesting approval. |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Meta-janitor safety approval for 5661121. Signal: #8900 (comment). GitHub auto-merge remains responsible for merging.
This was referenced Oct 27, 2026
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.
(janitor/comments): admin console comment cleanup
Package / topic area cleaned
The admin console module:
apps/web/src/app/admin/(Kilo Cloud admin UI — pages, API routes, and shared components under one app router tree).Files touched
48 files, all under
apps/web/src/app/admin/.Comment lines removed
272 comment lines (plus 34 directly-resulting blank lines; 1 line is a trailing-inline-comment removal on otherwise-unchanged code). 306 deletions, 1 insertion total — the insertion is the same code line re-emitted after its trailing comment was stripped; no code, types, tests, or assertions changed.
Categories of noise removed
// --- X ---,// ── X ──,// ----------)// Step 1: Stop if not already stopped,// First create the admin note,// Then block the user)// Get visible columns,// Filter by availability,// Handle sorting,// Validate tier)useStateclusters (// Credit form state,// API state,// Results state)classifyRecordtooltip body andisLikelyAbusecolor branches inUserAdminHeuristicAbuse.tsx)// Status and email badges)Important comments intentionally retained
csvExport.ts, SQL identifier validation inkiloclaw-analytics/route.ts, admin-audit-entry refetch avoidance insession-traces/hooks.ts)transferred_to_subscription_id, tier name mapping inOssSponsorshipDialog, inversepersonal_account_disabledflag, tri-state match state, dirty-form sync guards inBenchmarksSection)BeadInspectorDashboard, HTTP analytics middleware gap inkiloclaw-analytics)eslint-disabledirectives indebug/trpc/TrpcDebug.tsx(suppressions left untouched)Follow-up refactor candidates
None required — every deleted comment's information was fully recoverable from adjacent code.
Validation performed
git diff --checkcleanoxlintonapps/web/src/app/admin: pass (exit 0, no findings)tsgo --noEmitforapps/web: passjest src/app/admin/data-exports/data-export-components.test.ts: 9/9 fail with anAggregateErrorincleanupDbForTest/DB setup — identical on the clean tree (pre-existing sandbox environment limitation, unrelated to this change)