Fix: move files

[kaeizen]

Summary by CodeRabbit

• New Features
  • None
• Refactor
  • Centralized meta separator definitions to a single utility for consistent display across posts and the design library.
  • Reorganized style utility imports to improve maintainability without changing behavior.
• Chores
  • Consolidated and cleaned up public exports to reduce redundancy and clarify API surface.
• Notes
  • No functional changes expected; UI behavior and styling remain the same.

[coderabbitai]

Walkthrough

Centralizes META_SEPARATORS by introducing it in src/util/blocks.js, updating imports in dependent modules, and removing its local declaration/export from src/block/posts/util.js. Relocates blockStyleGenerators usage to a local module by removing its re-export from src/util/styles/index.js and importing it directly in the style guide utility.

Changes

Cohort / File(s)	Summary of changes
Meta separators centralization src/block/posts/util.js, src/components/design-library-list/util.js, src/util/blocks.js	Added exported constant META_SEPARATORS in src/util/blocks.js. Updated imports to source META_SEPARATORS from ~stackable/util. Removed local declaration/export of META_SEPARATORS from src/block/posts/util.js.
Style generators import relocation src/lazy-components/style-guide/utils.js, src/util/styles/index.js	Stopped re-exporting blockStyleGenerators from src/util/styles/index.js. Added direct import of blockStyleGenerators from ./block-style-generators in the style guide utils; removed prior import from ~stackable/util. No runtime logic changes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

A rabbit taps its paw in code delight,
Paths hop paths, now tidy and light.
Dots, dashes, pipes aligned in a row,
Styles burrow local, ready to go.
Thump-thump—refs are neat and terse,
Boundaries clear across the verse. 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title "Fix: move files" accurately reflects the primary intent of this changeset—relocating utility code and updating imports—because the diffs show META_SEPARATORS and blockStyleGenerators being moved and import paths adjusted; it is concise and not misleading.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/move-files-to-style-guide-component

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

🤖 Pull request artifacts

file	commit
pr3595-stackable-3595-merge.zip	341f8c5

[github-actions]

Try this Pull Request in the WordPress playground: https://playground.wordpress.net/?mode=seamless#{"landingPage":"/wp-admin/post-new.php?post_type=page","preferredVersions":{"php":"latest","wp":"latest"},"steps":[{"step":"login","username":"admin","password":"password"},{"step":"installPlugin","pluginZipFile":{"resource":"url","url":"https://raw.githubusercontent.com/gambitph/Stackable/artifacts/pr3595-stackable-3595-merge.zip"}}]}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

src/lazy-components/style-guide/utils.js (1)

3-4: Import relocation verified — no stale imports; named export present

No imports of blockStyleGenerators remain from '~stackable/util'. The local module exists and exports the named export. Files: src/lazy-components/style-guide/utils.js (import at line 3; use at line 213), src/lazy-components/style-guide/block-style-generators.js (export const blockStyleGenerators at line 18). Optional: guard against missing generators before using blockStyleGenerator (e.g. if (!blockStyleGenerator) return block).

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ff366d7 and 591fd50.

📒 Files selected for processing (5)

• src/block/posts/util.js (1 hunks)
• src/components/design-library-list/util.js (1 hunks)
• src/lazy-components/style-guide/utils.js (1 hunks)
• src/util/blocks.js (1 hunks)
• src/util/styles/index.js (0 hunks)

💤 Files with no reviewable changes (1)

• src/util/styles/index.js

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (7)

• GitHub Check: PHP 7.3 and WP latest
• GitHub Check: PHP 8.2 and WP 6.7.2
• GitHub Check: PHP 8.2 and WP latest
• GitHub Check: build
• GitHub Check: PHP 8.2 and WP 6.5.5
• GitHub Check: PHP 7.3 and WP 6.5.5
• GitHub Check: PHP 8.2 and WP 6.6.2

🔇 Additional comments (3)

src/components/design-library-list/util.js (1)

7-7: Switched META_SEPARATORS import to central util — verified OK.
No remaining imports from '~stackable/block/posts/util'; META_SEPARATORS exported at src/util/blocks.js:623 and imported at src/components/design-library-list/util.js:7 and src/block/posts/util.js:15.

src/util/blocks.js (1)

622-629: Freeze META_SEPARATORS to prevent accidental mutation

Wrap the exported object with Object.freeze; repo-wide verification failed in the sandbox—verify there are no other definitions or property writes before merging.

Apply:

-export const META_SEPARATORS = {
+export const META_SEPARATORS = Object.freeze({
 	dot: '·',
 	space: ' ',
 	comma: ',',
 	dash: '—',
 	pipe: '|',
-}
+})

src/block/posts/util.js (1)

15-15: Importing META_SEPARATORS from '~stackable/util' — approved.
META_SEPARATORS is defined and exported in src/util/blocks.js (export const META_SEPARATORS); no re-export found in src/block/posts/util.js.