Skip to content

Publicar @fiscalapi/docs-mcp en npm al fusionar en main (versión 4.0.461) - #3

Merged
mendozagit merged 1 commit into
mainfrom
ci/publish-docs-mcp
Oct 23, 2026
Merged

mendozagit merged 1 commit into
mainfrom
ci/publish-docs-mcp

Conversation

@mendozagit

Copy link
Copy Markdown
Contributor

Resumen

Agrega la publicación del servidor MCP en npm. Este commit quedó fuera de #2 porque llegó después de que se fusionó.

Cambios

  • Workflow Publish docs MCP en push a main con cambios en docs-mcp/, y también manual:
    • publica la versión de docs-mcp/package.json con el NPM_TOKEN de la organización y --provenance;
    • si esa versión ya está en npm, no publica;
    • antes de publicar corre typecheck, pruebas, build, build:data y verify:package.
  • La versión de @fiscalapi/docs-mcp es la 4.0.461, la misma de la API que documenta.
  • verify:package comprueba que el paquete incluya dist/ y data/ sin volver a correr prepack. Lo usan el CI y la publicación.
  • CI: el checkout ya no guarda las credenciales (persist-credentials: false).

Verificación

  • npm run typecheck y npm test: 178 pruebas en verde.
  • npm run verify:package: @fiscalapi/docs-mcp@4.0.461 con 120 archivos y 100 contratos de operación.

- Workflow Publish docs MCP en push a main (cambios en docs-mcp/) y manual. Publica con NPM_TOKEN de la organización y --provenance. Si la versión ya existe en npm, no publica.
- La versión del paquete es la de la API que documenta: 4.0.461.
- verify:package comprueba que el paquete trae dist/ y data/ sin repetir prepack. Lo usan el CI y la publicación.
- CI: checkout sin persist-credentials.
- Verificado: typecheck, 178 pruebas en verde y verify:package (@fiscalapi/docs-mcp@4.0.461 con 120 archivos y 100 contratos).
@coderabbitai

coderabbitai Bot commented Oct 23, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 31 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ 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: 9e328dfd-db31-4bca-844f-cd6e654412bf
📥 Commits

Reviewing files that changed from the base of the PR and between 5159ce0 and b9a881f.

⛔ Files ignored due to path filters: 1
  • docs-mcp/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing: 5
  • .github/workflows/check.yml
  • .github/workflows/publish-docs-mcp.yml
  • CONTRIBUTING.md
  • docs-mcp/package.json
  • docs-mcp/scripts/verify-package.ts
  • 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

Comment @coderabbitai help to get the list of available commands.

@mendozagit
mendozagit merged commit 89072ee into main Oct 23, 2026
3 checks passed
@mendozagit
mendozagit deleted the ci/publish-docs-mcp branch October 23, 2026 04:54
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