Skip to content

Add GET /moderation/all-extensions for a moderator's full extension list - #216

Merged
admdly merged 2 commits into
mainfrom
moderator-all-extensions-list
Sep 4, 2026
Merged

Add GET /moderation/all-extensions for a moderator's full extension list#216
admdly merged 2 commits into
mainfrom
moderator-all-extensions-list

Conversation

@admdly

@admdly admdly commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

What

Adds GET /moderation/all-extensions — lists every extension across every developer regardless of status, filterable by:

  • status: published | delisted | unpublished (omitted means all)
  • q: case-insensitive substring match on the extension id

Distinct from the existing GET /moderation/extensions, which lists revisions awaiting review — an extension's own status and its having a pending edit are independent.

Lists every extension across every developer, regardless of status,
filterable by status (published/delisted/unpublished) and by a
case-insensitive substring match on id (q). Distinct from the existing
GET /moderation/extensions, which lists revisions awaiting review -
an extension's own status and its having a pending edit are
independent.
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 4, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
api 030b6a1 Commit Preview URL

Branch Preview URL
Sep 04 2026, 12:37 PM

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 5 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread test/services/extensions/v2/moderation.test.ts
Comment thread src/services/extensions/v2/db/extensions.ts Outdated
…es, cover the _ wildcard case

toLowerCase() in JS is Unicode-aware while SQLite's LOWER() only folds
ASCII, so pre-lowering the search term could desync from LOWER(id) for
a non-ASCII id. Wrapping the term in LOWER() too keeps both sides
folded identically. Also adds the missing '_' assertion the test's own
title already claimed to cover.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

0 issues found across 2 files (changes from recent commits).

Requires human review: Adds a new moderator API endpoint to list all extensions across statuses, expanding the API surface and moderator data exposure; it is a new feature rather than a fix, requiring human sign-off on the intended scope.

Re-trigger cubic

@admdly
admdly merged commit c8e79f6 into main Sep 4, 2026
9 checks passed
@admdly
admdly deleted the moderator-all-extensions-list branch September 4, 2026 12:44
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