Skip to content

fix(ai-proxy): re-export FOREST_INTEGRATION_NAMES from package index [PRD-375]#1589

Merged
matthv merged 1 commit into
mainfrom
fix/prd-375-export-forest-integration-names-from-index
May 20, 2026
Merged

fix(ai-proxy): re-export FOREST_INTEGRATION_NAMES from package index [PRD-375]#1589
matthv merged 1 commit into
mainfrom
fix/prd-375-export-forest-integration-names-from-index

Conversation

@matthv
Copy link
Copy Markdown
Member

@matthv matthv commented May 20, 2026

Summary

Follow-up to #1588 (released as ai-proxy@1.10.0). The const FOREST_INTEGRATION_NAMES was declared and exported from forest-integration-client.ts but the package's src/index.ts has a selective named re-export block that did not include it. Consumers importing from @forestadmin/ai-proxy get undefined at runtime and any at the type level — the const is effectively invisible from the package boundary.

Change

```diff
export {

  • FOREST_INTEGRATION_NAMES,
    ForestIntegrationConfig,
    CustomConfig,
    ForestIntegrationName,
    } from './forest-integration-client';
    ```

How this slipped through 1.10.0

The original PR added tests inside ai-proxy that touched the file directly, so the internal declaration looked fine. No test asserted the visibility from the package's main entry. Caught only when wiring up the first consumer (forestadmin-server PR #8249) — the yarn upgrade to 1.10.0 + the import { FOREST_INTEGRATION_NAMES } swap failed lint and typecheck with any errors, which surfaced the missing re-export.

Test plan

  • `yarn workspace @forestadmin/ai-proxy build` → tsc clean
  • `yarn workspace @forestadmin/ai-proxy test` → 355 pass, 0 fail
  • `npx eslint packages/ai-proxy/src/index.ts` → clean
  • Verified generated `dist/index.d.ts` re-exports the const: `export { FOREST_INTEGRATION_NAMES, ... }`
  • Verified generated `dist/index.js` re-exports the value via `Object.defineProperty(exports, "FOREST_INTEGRATION_NAMES", ...)`

Release

Patch release (1.10.1) appropriate — fixes the export visibility intended in 1.10.0, no behavioral change for consumers already importing the type.

relates to PRD-375

🤖 Generated with Claude Code

Note

Re-export FOREST_INTEGRATION_NAMES from ai-proxy package index

Adds FOREST_INTEGRATION_NAMES to the named exports in index.ts, alongside the existing ForestIntegrationConfig, CustomConfig, and ForestIntegrationName exports from ./forest-integration-client.

🖇️ Linked Issues

Resolves PRD-375.

Macroscope summarized 80f7257.

…[PRD-375]

The const was declared and exported from forest-integration-client.ts in
1.10.0, but the package's index.ts has a selective re-export block that
did not include it. Consumers importing from '@forestadmin/ai-proxy' got
undefined at runtime and any at types.

This adds the missing re-export so the runtime const is accessible
through the package's main entry point, completing the work started in
1.10.0.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@linear
Copy link
Copy Markdown

linear Bot commented May 20, 2026

PRD-375

@matthv matthv marked this pull request as ready for review May 20, 2026 10:05
@qltysh
Copy link
Copy Markdown

qltysh Bot commented May 20, 2026

Qlty


Coverage Impact

This PR will not change total coverage.

Modified Files with Diff Coverage (1)

RatingFile% DiffUncovered Line #s
Coverage rating: A Coverage rating: A
packages/ai-proxy/src/index.ts100.0%
Total100.0%
🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@matthv matthv merged commit 909d050 into main May 20, 2026
29 checks passed
@matthv matthv deleted the fix/prd-375-export-forest-integration-names-from-index branch May 20, 2026 10:07
forest-bot added a commit that referenced this pull request May 20, 2026
## @forestadmin/ai-proxy [1.10.1](https://github.com/ForestAdmin/agent-nodejs/compare/@forestadmin/ai-proxy@1.10.0...@forestadmin/ai-proxy@1.10.1) (2026-05-20)

### Bug Fixes

* **ai-proxy:** re-export FOREST_INTEGRATION_NAMES from package index [PRD-375] ([#1589](#1589)) ([909d050](909d050))
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.

2 participants