Finish renaming the plugin #32

RadoslavGeorgiev · 2025-10-31T11:21:09Z

Description

There were a few remaining instances of names starting with OpenAi, including the main plugin file.

This PR addresses them.

Testing instructions

Activate the plugin and check if there are any fatal errors.
npm install & composer install should not be broken.
npm scripts, including build should load the new main file.

Summary by CodeRabbit

Documentation
- Updated plugin description to emphasize product feed functionality.
Chores
- Renamed package and plugin branding from OpenAI-Product-Feed to Product-Feed-for-OpenAI.
- Updated public package metadata and repository references.
Build / Tests
- Adjusted build and test setup to load and package the renamed plugin entry.

coderabbitai · 2025-10-31T11:21:27Z

Caution

Review failed

The pull request is closed.

Walkthrough

Rename and rebrand project artifacts: update main plugin filename references, package metadata (name/title/repository), test bootstrap path, and minor PHPDoc and plugin header text across build script, tests, source, and package files.

Changes

Cohort / File(s)	Summary
Build script `bin/build-prod-zip.sh`	Changed copied main plugin filename from `./openai-product-feed-for-woo.php` to `./woocommerce-product-feed-for-openai.php`.
Test bootstrap `tests/unit/bootstrap.php`	Updated mu-plugins load path from `/openai-product-feed-for-woo.php` to `/woocommerce-product-feed-for-openai.php`.
Package metadata `package.json`	Renamed package `@woocommerce/OpenAI-Product-Feed` → `@woocommerce/Product-Feed-for-OpenAI`, updated `title`, and changed `repository.url` to the woocommerce-product-feed-for-openai repo URL.
Plugin source & header `src/Core/Plugin.php`, `woocommerce-product-feed-for-openai.php`	Shortened PHPDoc description in `Plugin.php` and updated plugin header Description (removed "AI-optimized" wording) in `woocommerce-product-feed-for-openai.php`.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Pay attention to consistency of filename/path changes across build and test scripts.
Verify package.json rename matches distribution and registry expectations.
Confirm plugin header still contains all required WordPress header fields and correct slug/file name.

Possibly related PRs

Finish renaming the plugin #32 — Performs the same renaming of main plugin filename references, package.json updates, and test bootstrap changes.

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The PR title "Finish renaming the plugin" directly and clearly summarizes the primary change in the changeset. The changes across multiple files (build script, package.json, plugin main file, tests, and plugin header) all center on completing a renaming from OpenAI-based naming to the new "WooCommerce-Product-Feed" naming. The title accurately reflects this main objective without being vague or misleading.
Description Check	✅ Passed	The PR description includes the required Description and Testing instructions sections and adequately explains what the PR addresses (remaining OpenAI-based naming instances) and provides concrete testing steps (activation check, npm/composer verification, build script verification). However, the description is missing the Checklist section from the template, which should include checkboxes for `npm run test:php` and `npm run lint:php` verification. The core content areas are properly addressed, though the description doesn't fully adhere to the prescribed template structure.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between aca319f and 2765e55.

⛔ Files ignored due to path filters (1)

package-lock.json is excluded by !**/package-lock.json

📒 Files selected for processing (1)

package.json (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

RadoslavGeorgiev · 2025-10-31T11:21:51Z

package.json

 	"repository": {
 		"type": "git",
-		"url": "https://github.com/woocommerce/OpenAI-Product-Feed.git"
+		"url": "https://github.com/woocommerce/WooCommerce-Product-Feed.git"


This is not a typo, I just don't know what the long-term repository will be. This will not be published in NPM, so we should be good.

We can also the GitHub repo name too. Anyway, I prefer all characters to be lowercase: woocommerce-product-feed.

htdat

I think we should change and make it generic WooCommerce Product Feed for everything because this repo is now actually supporting product feeds for both POS and OpenAI specs.

WDYT @ebinnion ?

htdat · 2025-11-03T07:05:02Z

bin/build-prod-zip.sh

 FILES_TO_COPY=(
    "./src"
-    "./openai-product-feed-for-woo.php"
+    "./woocommerce-product-feed-for-openai.php"


May not use for-openai?

Suggested change

"./woocommerce-product-feed-for-openai.php"

"./woocommerce-product-feed.php"

I think a big part of the current code here will go into Woo Core at some point, and this plugin will remain OpenAI-specific.

package.json

coderabbitai

Actionable comments posted: 0

🧹 Nitpick comments (1)

tests/unit/bootstrap.php (1)
6-6: Fix typo in comment.

The comment contains "constnat" instead of "constant".

Apply this diff to fix the typo:
-// Define a constnat to use in tests.
+// Define a constant to use in tests.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9ac9d78 and aca319f.

📒 Files selected for processing (2)

src/Core/Plugin.php (1 hunks)
tests/unit/bootstrap.php (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

src/Core/Plugin.php

🧰 Additional context used

📓 Path-based instructions (1)

**/*.php

📄 CodeRabbit inference engine (.cursorrules)

When suggesting with Tab or writing code, always obey the WordPress Coding Standards (project PHPCS config in phpcs.xml.dist)

Files:

tests/unit/bootstrap.php

🧠 Learnings (2)

📓 Common learnings

Learnt from: RadoslavGeorgiev
Repo: woocommerce/OpenAI-Product-Feed PR: 13
File: src/API/Controllers/ApiController.php:44-50
Timestamp: 2025-10-22T15:49:57.065Z
Learning: The ApiController in src/API/Controllers/ApiController.php is planned to be moved and refactored to be OpenAI platform-specific, which is why it currently requires concrete ProductMapper and FeedValidator implementations from the OpenAI platform rather than interface types.

📚 Learning: 2025-10-24T17:54:03.161Z

Learnt from: RadoslavGeorgiev
Repo: woocommerce/OpenAI-Product-Feed PR: 21
File: src/Utils/MemoryManager.php:35-60
Timestamp: 2025-10-24T17:54:03.161Z
Learning: In the woocommerce/OpenAI-Product-Feed repository, the MemoryManager::flush_caches() method uses aggressive cache clearing including direct $wp_object_cache property manipulation. This is an intentional, tried & tested pattern copied from another project that is necessary for generating very large product feeds (70k-200k products). Without this aggressive approach, large exports fail. The cache clearing happens after feed processing is complete, before shipping the file to another server, not during normal page execution.

Applied to files:

tests/unit/bootstrap.php

🔇 Additional comments (1)

tests/unit/bootstrap.php (1)

33-33: LGTM! Plugin file path updated correctly.

The path change aligns with the renaming effort described in the PR objectives and is consistent with the updates in other files (e.g., bin/build-prod-zip.sh).

RadoslavGeorgiev · 2025-11-03T09:16:25Z

I think we should change and make it generic WooCommerce Product Feed for everything because this repo is now actually supporting product feeds for both POS and OpenAI specs.

WDYT @ebinnion ?

As I mentioned in this comment, I think that mostly Integrations/OpenAi will remain within this plugin in the near future.

ebinnion · 2025-11-03T16:47:42Z

I think that it'll largely be OpenAI. For POS, I think that we'll want to pull stuff to Woo core.

RadoslavGeorgiev · 2025-11-03T18:14:59Z

If you see this message, please run this command :)

git remote set-url origin [email protected]:woocommerce/woocommerce-product-feed-for-openai.git

Rename the remaining instances, starting with OpenAI

9ac9d78

RadoslavGeorgiev commented Oct 31, 2025

View reviewed changes

RadoslavGeorgiev requested a review from htdat October 31, 2025 11:24

RadoslavGeorgiev marked this pull request as ready for review October 31, 2025 11:24

htdat reviewed Nov 3, 2025

View reviewed changes

Merge branch 'trunk' into wooptp-34-rename-product-feed-extension

aca319f

coderabbitai bot reviewed Nov 3, 2025

View reviewed changes

A bit of consistency with the -for-openai part

08c962d

Fixing a misspelled WOo

2765e55

RadoslavGeorgiev merged commit 6a1b3e8 into trunk Nov 3, 2025
1 check was pending

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Finish renaming the plugin #32

Finish renaming the plugin #32

RadoslavGeorgiev commented Oct 31, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

coderabbitai bot commented Oct 31, 2025 •

edited

Loading

Review failed

Uh oh!

RadoslavGeorgiev Oct 31, 2025

Uh oh!

htdat Nov 3, 2025

Uh oh!

htdat left a comment

Uh oh!

htdat Nov 3, 2025

Uh oh!

RadoslavGeorgiev Nov 3, 2025

Uh oh!

Uh oh!

coderabbitai bot left a comment

Uh oh!

RadoslavGeorgiev commented Nov 3, 2025

Uh oh!

ebinnion commented Nov 3, 2025

Uh oh!

Uh oh!

RadoslavGeorgiev commented Nov 3, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

	"./woocommerce-product-feed-for-openai.php"
	"./woocommerce-product-feed.php"

Finish renaming the plugin #32

Finish renaming the plugin #32

Conversation

RadoslavGeorgiev commented Oct 31, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Testing instructions

Summary by CodeRabbit

Uh oh!

coderabbitai bot commented Oct 31, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review failed

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Pre-merge checks and finishing touches

Uh oh!

RadoslavGeorgiev Oct 31, 2025

Choose a reason for hiding this comment

Uh oh!

htdat Nov 3, 2025

Choose a reason for hiding this comment

Uh oh!

htdat left a comment

Choose a reason for hiding this comment

Uh oh!

htdat Nov 3, 2025

Choose a reason for hiding this comment

Uh oh!

RadoslavGeorgiev Nov 3, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

RadoslavGeorgiev commented Nov 3, 2025

Uh oh!

ebinnion commented Nov 3, 2025

Uh oh!

Uh oh!

RadoslavGeorgiev commented Nov 3, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

RadoslavGeorgiev commented Oct 31, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Oct 31, 2025 •

edited

Loading