Skip to content

Cleanups - #18

Merged
nmbrthirteen merged 1 commit into
mainfrom
cleanups
Nov 3, 2026
Merged

nmbrthirteen merged 1 commit into
mainfrom
cleanups

Conversation

@nmbrthirteen

Copy link
Copy Markdown
Owner

What changed

  • Publishing holds until a real AI provider is configured. The mock provider runs in tests only.
  • Fact checks run before a post or update goes live.
  • Entity matching ignores sentence openers and needs two shared facts to merge.
  • Embedding fallbacks are reported.
  • Dedup uses stored embeddings and the configured threshold.
  • Updates add only new facts.
  • IndexNow calls are checked. The keyword provider filter is wired in.
  • API keys use sodium and the Gemini key moves to a header.
  • README matches the WP-Cron queue.
  • Removes em and en dashes and uppercase labels.

How I checked it

  • php -l passes on every changed file.
  • The test suite passes 101 of 101 through a PHPUnit stand-in. composer could not download here.

Needs attention

  • With no provider configured, imports wait instead of publishing. Existing sites still on the mock provider need a key.
  • Stored API keys are re-encrypted on the next settings save.

- Publishing holds until a real AI provider is configured. The mock provider runs in tests only.
- Fact checks run before a post or update goes live.
- Entity matching ignores sentence openers and needs two shared facts to merge.
- Embedding fallbacks are reported.
- Dedup uses stored embeddings and the configured threshold.
- Updates add only new facts.
- IndexNow calls are checked. The keyword provider filter is wired in.
- API keys use sodium and the Gemini key moves to a header.
- README matches the WP-Cron queue.
- Removes em and en dashes and uppercase labels.
@coderabbitai

coderabbitai Bot commented Nov 3, 2026

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 0ad88cfa-e241-44fb-8376-756befaf30e0
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.

@nmbrthirteen
nmbrthirteen merged commit 20b3407 into main Nov 3, 2026
6 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.

1 participant