feat: stdをインタプリタから分離 by uzmoi · Pull Request #1022 · aiscript-dev/aiscript

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@uzmoi

Copy link

Member

@uzmoi uzmoi commented

Jan 14, 2026

edited

Loading

What

インタプリタがデフォルトではprintとreadlineとCore名前空間内のもの以外のグローバル変数を定義しなくしました。
stdのエクスポート形式や粒度は要検討。
Coreはインタプリタと密結合(二項演算子など)なため引き続き上書き不可能な形で定義しています。

Why

Resolve #671

Additional info (optional)

FineArchs and takejohn reacted with thumbs up emoji

@codecov-commenter

Copy link

codecov-commenter commented

Jan 17, 2026

edited

Loading

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 89.92248% with 13 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/interpreter/lib/core.ts 88.59% 13 Missing ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Files with missing lines Coverage Δ
src/index.ts 100.00% <100.00%> (ø)
src/interpreter/index.ts 72.11% <100.00%> (ø)
src/interpreter/lib/std.ts 82.92% <100.00%> (ø)
src/interpreter/lib/core.ts 88.59% <88.59%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@uzmoi uzmoi marked this pull request as ready for review

January 17, 2026 23:17
Copy link

Member

@FineArchs FineArchs left a comment

Choose a reason for hiding this comment

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

ホスト向けの変更なので、For Hosts:としてChangelogを書いておいてください。

@uzmoi

Copy link

Member Author

uzmoi commented

Jan 20, 2026

忘れてた……追加しました

FineArchs reacted with heart emoji

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@FineArchs FineArchs FineArchs approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@uzmoi @codecov-commenter @FineArchs