Skip to content

Fix package metadata updates on republish#269

Merged
khaliqgant merged 2 commits into
mainfrom
codex/fix-republish-package-metadata
May 13, 2026
Merged

Fix package metadata updates on republish#269
khaliqgant merged 2 commits into
mainfrom
codex/fix-republish-package-metadata

Conversation

@khaliqgant

@khaliqgant khaliqgant commented May 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • update mutable package metadata from the latest manifest when republishing an existing package
  • mark packages for AI re-enrichment when republished metadata changes
  • add a regression test covering generic-to-claude republish metadata drift
  • fix README package-count typo in root and CLI docs

Closes #259

Tests

  • npm exec --workspace=@pr-pm/registry -- vitest run src/routes/tests/packages.test.ts
  • npm run typecheck --workspace=@pr-pm/registry

@codeant-ai

codeant-ai Bot commented May 13, 2026

Copy link
Copy Markdown

CodeAnt AI is reviewing your PR.


Thanks for using CodeAnt! 🎉

We're free for open-source projects. if you're enjoying it, help us grow by sharing.

Share on X ·
Reddit ·
LinkedIn

@codeant-ai codeant-ai Bot added the size:L This PR changes 100-499 lines, ignoring generated files label May 13, 2026
@my-senior-dev-pr-review

my-senior-dev-pr-review Bot commented May 13, 2026

Copy link
Copy Markdown

🤖 My Senior Dev — Analysis Complete

👤 For @khaliqgant

📁 Expert in packages/ (231 edits) • ⚡ 2nd PR this month

View your contributor analytics →


📊 4 files reviewed • 3 high risk • 5 need attention

🚨 High Risk:

  • packages/registry/src/routes/packages.ts — This file introduces critical changes to the update query affecting data integrity and security. The missing validation and authorization checks are significant risks.

⚠️ Needs Attention:

  • README.md — Only contains trivial changes but should still be noted for completeness.

🚀 Open Interactive Review →

The full interface unlocks features not available in GitHub:

  • 💬 AI Chat — Ask questions on any file, get context-aware answers
  • 🔍 Smart Hovers — See symbol definitions and usage without leaving the diff
  • 📚 Code Archeology — Understand how files evolved over time (/archeology)
  • 🎯 Learning Insights — See how this PR compares to similar changes

💬 Chat here: @my-senior-dev explain this change — or try @chaos-monkey @security-auditor @optimizer @skeptic @junior-dev

📖 View all 12 personas & slash commands

You can interact with me by mentioning @my-senior-dev in any comment:

In PR comments or on any line of code:

  • Ask questions about the code or PR
  • Request explanations of specific changes
  • Get suggestions for improvements

Slash commands:

  • /help — Show all available commands
  • /archeology — See the history and evolution of changed files
  • /profile — Performance analysis and suggestions
  • /expertise — Find who knows this code best
  • /personas — List all available AI personas

AI Personas (mention to get their perspective):

Persona Focus
@chaos-monkey 🐵 Edge cases & failure scenarios
@skeptic 🤨 Challenge assumptions
@optimizer Performance & efficiency
@security-auditor 🔒 Security vulnerabilities
@accessibility-advocate Inclusive design
@junior-dev 🌱 Simple explanations
@tech-debt-collector 💳 Code quality & shortcuts
@ux-champion 🎨 User experience
@devops-engineer 🚀 Deployment & scaling
@documentation-nazi 📚 Documentation gaps
@legacy-whisperer 🏛️ Working with existing code
@test-driven-purist Testing & TDD

For the best experience, view this PR on myseniordev.com — includes AI chat, file annotations, and interactive reviews.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 2 files

@codeant-ai

codeant-ai Bot commented May 13, 2026

Copy link
Copy Markdown

CodeAnt AI finished reviewing your PR.

@khaliqgant
khaliqgant merged commit d2db101 into main May 13, 2026
11 checks passed
@khaliqgant
khaliqgant deleted the codex/fix-republish-package-metadata branch May 13, 2026 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

typo in readme 7,5000

1 participant