[PR #206] [MERGED] fix: add missing SliderRenderable export #367

New issue

Closed

opened 2026-03-02 23:46:10 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 23:46:10 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/anomalyco/opentui/pull/206
Author: @thefinalb055
Created: 10/8/2025
Status: ✅ Merged
Merged: 10/12/2025
Merged by: @kommander

Base: main ← Head: fix/missing-renderable-exports

📝 Commits (4)

5143cc0 fix: add missing SliderRenderable export to @opentui/core
72b8a0a Merge branch 'main' into fix/missing-renderable-exports
1c27946 Merge branch 'main' into fix/missing-renderable-exports
f50d413 Prettier ran - formatted 1 file (index.ts)

📊 Changes

1 file changed (+10 additions, -9 deletions)

View changed files

📝 packages/core/src/renderables/index.ts (+10 -9)

📄 Description

Summary

Fixes #205

Adds the missing export * from "./Slider" to packages/core/src/renderables/index.ts.

Problem

The published @opentui/core@0.1.25 package was missing the SliderRenderable export, causing runtime errors when importing it outside the monorepo:

import { SliderRenderable } from "@opentui/core" // Error: not found

Solution

Added the missing export statement for Slider. Note that CodeRenderable and TextBufferRenderable exports were already added in PR #191, so this PR only addresses the Slider export.

Verification

Verified in the build output (bun run build:lib):

✅ dist/renderables/Slider.d.ts exists
✅ dist/renderables/index.d.ts includes export * from "./Slider"
✅ dist/index.js exports SliderRenderable in the main export block

Test Plan

Build succeeds (bun run build:lib)
Tests pass (bun run test:js - 658 pass)
SliderRenderable appears in dist/index.js exports
Type definitions generated correctly

🤖 Generated with Claude Code

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/anomalyco/opentui/pull/206 **Author:** [@thefinalb055](https://github.com/thefinalb055) **Created:** 10/8/2025 **Status:** ✅ Merged **Merged:** 10/12/2025 **Merged by:** [@kommander](https://github.com/kommander) **Base:** `main` ← **Head:** `fix/missing-renderable-exports` --- ### 📝 Commits (4) - [`5143cc0`](https://github.com/anomalyco/opentui/commit/5143cc0bff3dcd8e52ced949e7695b8f1de08255) fix: add missing SliderRenderable export to @opentui/core - [`72b8a0a`](https://github.com/anomalyco/opentui/commit/72b8a0a7bc0ba252efb67e17abf3cfc2437d0cd9) Merge branch 'main' into fix/missing-renderable-exports - [`1c27946`](https://github.com/anomalyco/opentui/commit/1c27946bf2c6abd641e58effd48f24aef14a2c64) Merge branch 'main' into fix/missing-renderable-exports - [`f50d413`](https://github.com/anomalyco/opentui/commit/f50d413b1ef952f7d6dbaf91dc770ebc3ef11673) Prettier ran - formatted 1 file (index.ts) ### 📊 Changes **1 file changed** (+10 additions, -9 deletions) <details> <summary>View changed files</summary> 📝 `packages/core/src/renderables/index.ts` (+10 -9) </details> ### 📄 Description ## Summary Fixes #205 Adds the missing `export * from "./Slider"` to `packages/core/src/renderables/index.ts`. ## Problem The published `@opentui/core@0.1.25` package was missing the `SliderRenderable` export, causing runtime errors when importing it outside the monorepo: ```typescript import { SliderRenderable } from "@opentui/core" // Error: not found ``` ## Solution Added the missing export statement for Slider. Note that `CodeRenderable` and `TextBufferRenderable` exports were already added in PR #191, so this PR only addresses the Slider export. ## Verification Verified in the build output (`bun run build:lib`): - ✅ `dist/renderables/Slider.d.ts` exists - ✅ `dist/renderables/index.d.ts` includes `export * from "./Slider"` - ✅ `dist/index.js` exports `SliderRenderable` in the main export block ## Test Plan - [x] Build succeeds (`bun run build:lib`) - [x] Tests pass (`bun run test:js` - 658 pass) - [x] SliderRenderable appears in dist/index.js exports - [x] Type definitions generated correctly 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>