Skip to content

fix: remove stale Wikidata facts, fix Meta AI valuation/cash-burn labels#3248

Merged
OAGr merged 2 commits intomainfrom
claude/fix-p2-factbase-cleanup
Mar 26, 2026
Merged

fix: remove stale Wikidata facts, fix Meta AI valuation/cash-burn labels#3248
OAGr merged 2 commits intomainfrom
claude/fix-p2-factbase-cleanup

Conversation

@OAGr
Copy link
Copy Markdown
Contributor

@OAGr OAGr commented Mar 26, 2026

Summary

Fixes 4 P2 bugs from QA sweep (Discussion #3220):

  • Bug 47: Removed AMI Labs $3.5B valuation from Meta AI entity (wrong entity)
  • Bug 48: Renamed cash-burncumulative-losses for Meta AI's $70B Reality Labs figure (was mislabeled as annual)
  • Bug 56: Removed 5 stale Wikidata-imported facts (Anthropic headcount=500, OpenAI headcount=375, OpenAI revenue=ambiguous, OpenAI legal-structure=501(c)(3), DeepMind headcount=10000) — all superseded by sourced facts
  • Bug 57: Removed 2 duplicate facts (DeepMind founded-date, OpenAI total-funding)

7 stale/incorrect facts removed, 1 property renamed.

Test plan

  • No code changes — data-only YAML edits
  • Facts validated against existing sourced entries

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Removed outdated fact entries (headcount, funding, revenue, founding dates) from company profiles to improve data consistency.
    • Updated Reality Labs financial metric classification from cash-burn to cumulative-losses with clarified documentation.
    • Cleaned up unrelated description entries from company records.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 26, 2026

Warning

Rate limit exceeded

@OAGr has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 25 minutes and 18 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7d6e81ba-4de0-40f0-afbd-ab1c189da7f5

📥 Commits

Reviewing files that changed from the base of the PR and between 43296e5 and 2960ffc.

📒 Files selected for processing (5)
  • packages/factbase/data/properties.yaml
  • packages/factbase/data/things/anthropic.yaml
  • packages/factbase/data/things/deepmind.yaml
  • packages/factbase/data/things/meta-ai.yaml
  • packages/factbase/data/things/openai.yaml
📝 Walkthrough

Walkthrough

These changes remove and update facts from multiple YAML company data files within the factbase package. Specifically, Wikidata-sourced facts are removed from Anthropic, DeepMind, and OpenAI records. A Reality Labs financial property in Meta AI's data is updated from cash-burn to cumulative-losses, with supporting documentation refined.

Changes

Cohort / File(s) Summary
Company Factbase Data
packages/factbase/data/things/anthropic.yaml, packages/factbase/data/things/deepmind.yaml, packages/factbase/data/things/meta-ai.yaml, packages/factbase/data/things/openai.yaml
Removed Wikidata-sourced facts (headcount, founded-date, total-funding, revenue) from Anthropic, DeepMind, and OpenAI. Updated Meta AI's Reality Labs fact property from cash-burn to cumulative-losses and removed an unrelated description entry.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

stage:approved

Poem

🐰 Facts were tidied, duplicates shed,
Wikidata whispers laid to rest instead,
Meta's burns became losses clear,
The factbase now rings bright and dear! ✨

🚥 Pre-merge checks | ✅ 3
✅ 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 accurately summarizes the main changes: removing stale Wikidata facts and correcting Meta AI property labels, matching the PR's primary objectives.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/fix-p2-factbase-cleanup

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.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@packages/factbase/data/things/meta-ai.yaml`:
- Around line 69-74: Add a new property definition for "cumulative-losses" to
the properties schema so the YAML entry in
packages/factbase/data/things/meta-ai.yaml validates and renders correctly: open
packages/factbase/data/properties.yaml and add an entry keyed
"cumulative-losses" mirroring the financial formatting used by "cash-burn" (use
the same type/format/units/label fields as "cash-burn" to ensure currency/number
formatting and display), include a brief description and any relevant aliases,
and commit the change; ensure the property name exactly matches
"cumulative-losses".
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 61457fd7-163d-42ca-9949-8fd46c85f454

📥 Commits

Reviewing files that changed from the base of the PR and between 8b8627c and 43296e5.

📒 Files selected for processing (4)
  • packages/factbase/data/things/anthropic.yaml
  • packages/factbase/data/things/deepmind.yaml
  • packages/factbase/data/things/meta-ai.yaml
  • packages/factbase/data/things/openai.yaml
💤 Files with no reviewable changes (3)
  • packages/factbase/data/things/deepmind.yaml
  • packages/factbase/data/things/anthropic.yaml
  • packages/factbase/data/things/openai.yaml

Comment thread packages/factbase/data/things/meta-ai.yaml
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 26, 2026

Test Coverage Report

Workspace Lines Branches Functions Statements
wiki-server 50.8% 34.1% 47.3% 49.5%
web 9.7% 7.8% 5.6% 9.5%
discord-bot 39.4% 30.5% 44.1% 38.4%
crux 27.0% 24.2% 30.3% 27.1%
Total 25.0% 18.8% 20.9% 24.6%

Coverage is advisory only — no thresholds enforced. Generated by vitest --coverage.

@OAGr OAGr force-pushed the claude/fix-p2-factbase-cleanup branch 2 times, most recently from 9b42db9 to b9fd2bb Compare March 26, 2026 18:24
github-actions Bot and others added 2 commits March 26, 2026 14:48
Bug 47: Remove AMI Labs $3.5B valuation description from meta-ai.yaml — that
valuation belongs to Yann LeCun's separate AMI Labs startup, not Meta AI.

Bug 48: Rename cash-burn → cumulative-losses for Reality Labs $70B figure in
meta-ai.yaml; the note now explicitly states this is a ~5-year cumulative
total, not an annual figure.

Bug 56: Remove stale Wikidata-imported facts superseded by more recent data:
- anthropic.yaml: headcount=500 (CTxrfaCzbw); current value is 4,074
- openai.yaml: headcount=375 (JU8dVl7ViQ); current value is 4,500
- openai.yaml: revenue=5363248 (hf1lYXbYXQ); ambiguous unit/date, stale
- openai.yaml: legal-structure=501(c)(3) (Jjf6iAusCA); OpenAI is now a PBC
- deepmind.yaml: headcount=10000 (mZo3jiRTlQ); current value is 6,000

Bug 57: Remove duplicate facts:
- deepmind.yaml: duplicate founded-date 2010-01 (F2JXn1Rz4g) removed; keep
  the Wikipedia-sourced 2010-09 with specific month
- openai.yaml: duplicate total-funding=5.79e+10 at 2025-03 (f_m5b6R9pIby)
  removed; keep the first entry with the more detailed SoftBank round notes

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Mirrors cash-burn format (financial, USD, billions display).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@OAGr OAGr force-pushed the claude/fix-p2-factbase-cleanup branch from b9fd2bb to 2960ffc Compare March 26, 2026 18:48
@OAGr OAGr added this pull request to the merge queue Mar 26, 2026
Merged via the queue into main with commit 9d12d1f Mar 26, 2026
22 checks passed
@OAGr OAGr deleted the claude/fix-p2-factbase-cleanup branch March 26, 2026 19:02
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