Skip to content

feat!: remove OpenAI Moderation API Example#1655

Merged
YasharF merged 1 commit into
sahat:masterfrom
YasharF:removeOpenAIMod
May 23, 2026
Merged

feat!: remove OpenAI Moderation API Example#1655
YasharF merged 1 commit into
sahat:masterfrom
YasharF:removeOpenAIMod

Conversation

@YasharF
Copy link
Copy Markdown
Collaborator

@YasharF YasharF commented May 23, 2026

Checklist

  • I acknowledge that submissions that include copy-paste of AI-generated content taken at face value (PR text, code, commit message, documentation, etc.) most likely have errors and hence will be rejected entirely and marked as spam or invalid
  • I manually tested the change with a running instance, DB, and valid API keys where applicable
  • Added/updated tests if the existing tests do not cover this change
  • README or other relevant docs are updated
  • --no-verify was not used for the commit(s)
  • npm run lint passed locally without any errors
  • npm test passed locally without any errors
  • npm run test:e2e:replay passed locally without any errors
  • npm run test:e2e:custom -- --project=chromium-nokey-live passed locally without any errors
  • PR diff does not include unrelated changes
  • PR title follows Conventional Commits — https://www.conventionalcommits.org/en

Description

Fixes: #1597

Screenshots of UI changes (browser) and logs/test results (console, terminal, shell, cmd)

/ai view after the removal.
image

Copilot AI review requested due to automatic review settings May 23, 2026 04:38
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes the OpenAI Moderation API example from the AI section of the app, including its UI entrypoint, routes/controllers, documentation, and associated E2E/fixture coverage (per issue #1597).

Changes:

  • Removed /ai/openai-moderation UI page and its card from the /ai index.
  • Removed Express routes and controller handlers for the OpenAI moderation example.
  • Removed the Playwright E2E test and recorded fixtures, and updated testing/docs references accordingly.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
views/ai/openai-moderation.pug Removes the OpenAI Moderation example page template.
views/ai/index.pug Removes the OpenAI Moderation card/link from the AI examples index.
controllers/ai.js Removes the GET/POST controller actions for the moderation example.
app.js Removes the /ai/openai-moderation routes.
README.md Removes OpenAI Moderation mentions and setup instructions.
test/e2e/openai-moderation.e2e.test.js Removes the Playwright E2E test for the moderation example.
test/TESTING.md Removes the moderation E2E test from the documented test tree.
test/fixtures/POST_https%3A%2F%2Fapi.openai.com%2Fv1%2Fmoderations_c6b4d54f3bd4.json Removes recorded OpenAI moderation fixture (flagged case).
test/fixtures/POST_https%3A%2F%2Fapi.openai.com%2Fv1%2Fmoderations_624f7df3dc5f.json Removes recorded OpenAI moderation fixture (non-flagged case).
test/fixtures/fixture_manifest.json Removes the moderation test from the fixture manifest (plus a minor formatting-only reflow; see comment).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread test/fixtures/fixture_manifest.json
@YasharF YasharF merged commit 31fea30 into sahat:master May 23, 2026
7 of 8 checks passed
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.

Remove openAI moderation example

2 participants