[sfaqer]

Вычислить/Выполнить кэшируют скомпилированное выражение, а оно связывало области по номеру в списке кадра: сначала глобальные контексты, за ними область модуля и вложенных вычислений. ПодключитьВнешнююКомпоненту с [GlobalContext] добавляет в этот список контекст, номера сдвигаются, и выражение из кэша вместо переменной или метода модуля берет свойство/метод компоненты или падает с ArgumentOutOfRangeException. Воспроизводится и в одном потоке:

Для Сч = 1 По 2 Цикл
	Сообщить(Вычислить("МояПеременная")); // на втором круге - свойство компоненты
	Если Сч = 1 Тогда
		ПодключитьВнешнююКомпоненту(ПутьКDll); // с глобальным контекстом
	КонецЕсли;
КонецЦикла;

Глобальные контексты теперь связываются с выражением напрямую, а область модуля и вложенных вычислений — по номеру с конца списка, который от числа глобальных контекстов не зависит.

Тест EvalScopesTests в Core.Tests, а tests/global-context-addin.os прогоняет сценарии из #344 и #911 и подключение во время работы с тестовой компонентой, которая теперь добавляет глобальный контекст.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Bug Fixes
  • Corrected how scripts resolve module variables and functions after a global context is attached, including in cached expressions and nested execution.
  • Improved global-context access across component load orders, dynamically loaded scripts, and objects created before or after attachment.
  • Corrected how nested execution scopes are identified so references resolve to the intended scope.

[coderabbitai]

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

• Last reviewed commit: c4e786b

Use the following commands to manage reviews:

• @coderabbitai resume to resume automatic reviews.
• @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

• ▶️ Resume reviews
• 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: d201c23c-da2d-4583-8c9e-46bf59200a47

📥 Commits

Reviewing files that changed from the base of the PR and between 8135c04 and c4e786b.

📒 Files selected for processing: 3

• src/Tests/OneScript.Core.Tests/EvalScopesTests.cs
• tests/global-context-addin.os
• tests/global-context/nested-execute.os

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

📝 Walkthrough

Scope binding now separates root scopes from nested inner scopes. Compiler context extraction and frame-scope resolution use reverse indexing for inner scopes. Added tests cover evaluated expressions and global-context loading and attachment scenarios.

Changes: Scope Binding

Layer / File(s)	Summary
Scope mapping and resolution src/OneScript.Core/Compilation/Binding/ScopeBindingDescriptor.cs, src/ScriptEngine/Machine/JoinedScopes.cs, src/ScriptEngine/Machine/MachineInstance.cs, src/ScriptEngine/Machine/ModuleSymbolBinding.cs, src/ScriptEngine/Compiler/ModuleDumpWriter.cs	JoinedScopes tracks the root scope list and inner-scope count, and provides reverse lookup. Compiler context extraction adds root scopes with static bindings and inner scopes with reverse-indexed frame bindings. Frame-scope resolution uses reverse lookup, and module dumps show a caret-prefixed, one-based scope depth.
Evaluation scope validation src/Tests/OneScript.Core.Tests/EvalScopesTests.cs	Adds tests that attach a global context during a loop and during nested execution, then check evaluated expressions and module-scope values.
Global-context loading and attachment scenarios src/Component/SimpleGlobalContext.cs, tests/global-context-addin.os, tests/global-context/*	Adds a test component and scenarios for library load order, script loading, runtime attachment, nested execution, and objects created before component attachment. The add-in runs the scenarios in separate processes and checks their output.

Priority: ⬆️ High

Estimated code review effort: 3 (Moderate) | ~25 minutes

Change: Bug fix

Suggested reviewers: evilbeaver, mr-rm

Merge Risk: ⚪ Minimal · up to c4e78

The change adds tests for attaching a global-context component during nested execution. No merge-blocking problem was found in the supplied changes.

Security Architecture Review

Security architecture risk: 🔵 Low · up to 92241

The change stabilizes cached expression bindings without demonstrating broader access to objects or methods. Risk is bounded, but compatibility with external hosts and behavior during concurrent attachment or interrupted execution remain unverified.

Retained concerns
No architecture-level concerns identified.

Security review details

Security Blast Radius

• inferred — The demonstrated exposure is context selection within an execution machine: script expressions resolve properties and methods on existing global or frame contexts. The inspected change does not demonstrate an expansion to additional services, tenants, credentials, or data stores; downstream host exposure remains unverified.

Trust Boundaries and Controls

• inferred — For inspected callers, script-provided expressions continue through symbol extraction and binding resolution to existing context interfaces. Direct root bindings preserve the selected context identity, and bounded reverse lookup prevents root growth from redirecting an inner binding to a component context. No new authorization bypass was demonstrated by this flow.

Resilience and Maintainability Implications

• observed — Global contexts remain backed by an unsynchronized mutable list. Evaluate has finally-based frame cleanup, whereas Execute and debugger evaluation retain pre-existing cleanup paths reached after successful execution. These conditions were not shown to worsen in this PR, but concurrent attachment and interruption recovery remain unestablished for the changed binding lifecycle.

Pre-merge checks: 4 · 1

Failed checks: 1 warning

Check name	Status	Explanation	Resolution
Docstring Coverage		Docstring coverage is 11.76% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 17 functions across 7 files. (2 skipped: …	Write docstrings for the functions missing them to satisfy the coverage threshold.

Passed checks: 4 passed

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		Название точно описывает основное изменение: кэшированные Вычислить и Выполнить больше не зависят от количества глобальных контекстов.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

Docstring coverage is 11.76% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 17 functions across 7 files. (2 skipped: 2 unsupported.)

• Fix all pre-merge checks with AI

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[EvilBeaver]

Внешние компоненты, которые добавляют глобальный скоуп ривносят столько адовых ошибок, что мы просто отказались их поддерживать. На эту тему тут уже заведено три или больше issues. Если по простому, то добавление контекстов в рантайме дает очень и очень неочевидные баги.

Иными словами, я боюсь это трогать и вливать. Надо найти все старые issues с этими же проблемами и смотреть в т.ч. и те тестовые ситуации

[sfaqer]

Нашел старые: #344 и #911 (к ним привязан #804). Там компонента с глобальным контекстом подключалась в ПриЗагрузкеБиблиотеки, и если библиотека была не последней в #Использовать, движок падал с ArgumentOutOfRangeException в SetFrame или с NPE на вызове метода модуля. Это исправлено в #956.

Прогнал их сценарии на develop и на этой ветке с тестовой компонентой с [GlobalContext]: библиотека с компонентой первой и последней рядом с обычной библиотекой, вызов метода модуля, загрузка такого сценария через ЗагрузитьСценарий, как в testrunner. Обе сборки работают одинаково. Разница только при подключении во время работы: на develop Вычислить после ПодключитьВнешнююКомпоненту падает с ArgumentOutOfRangeException, на ветке переменные и методы модуля на месте, а Вычислить видит компоненту.

Если нужно, добавлю эти сценарии в тесты: класс с [GlobalContext] в тестовую компоненту и сценарии в tests/.

[sfaqer]

Добавил эти сценарии в тесты: в тестовой компоненте теперь есть класс с [GlobalContext], а tests/global-context-addin.os прогоняет сценарии из #344 и #911 и подключение во время работы, каждый отдельным процессом. На develop без исправления падает только подключение во время работы, остальные проходят и там.

[EvilBeaver]

Есть еще issue #1000 тоже связанный с порядком добавления глобальных скоупов.

[sfaqer]

Есть еще issue #1000 тоже связанный с порядком добавления глобальных скоупов.

С этим PR он не связан. #1000 про порядок выполнения тел модулей библиотеки: InitExternalLibrary выполнял их по порядку файлов, и тело модуля, обратившееся к модулю, до которого очередь еще не дошла, видело его переменные пустыми. Глобальных контекстов модули не добавляют, на develop, 2.2.0 и этой ветке поведение было одинаковым. Починил в #1788.

[EvilBeaver]

Мне кажется норм. Я бы попросил еще кого-то взглянуть, страшновато трогать это место. @Mr-Rm ?

[Mr-Rm]

Обозначенная проблема решается.
Проверить бы ещё со вложенными друг в друга Выполнить и ПодключитьВнешнююКомпоненту
Порядок добавления модулей в глобальный контекст и перекрытие методов - вне этого PR

[sfaqer]

@Mr-Rm Добавил: global-context/nested-execute.os с тестовой компонентой и NestedExecuteSeesModuleAfterGlobalContextIsAttachedInside в EvalScopesTests (в Jenkins компоненты нет, а этот идет и там). Внешний Выполнить подключает компоненту, следующий вложенный ее уже видит, на втором круге все из кэша. На develop оба падают с ArgumentOutOfRangeException на вложенном Выполнить сразу после подключения.

[sonar-openbsl-ru-qa-bot]

Quality Gate passed

Issues

• 0 New Issues
• 2 Fixed Issues
• 0 Accepted Issues

Measures

• 0 Security Hotspots
• No data about coverage (35.10% Estimated after merge)
• 0.00% Duplicated Code (2.50% Estimated after merge)

Project ID: OneScript

View in SonarQube