….502.5 (microsoft#296)

…oft#298)

The Tool Auto Approve page stacked four rule sections (~850px) in an
uncapped container, so JFace's PreferenceDialog grew the shell toward
screen height and never shrank it.

Wrap the sections in a height-capped ScrolledComposite whose computeSize
bounds the height reported to the dialog's PageLayout, so the shell stays
a normal size and the content scrolls within a single page-level
scrollbar.

Unify the cap as PreferencePageUtils.STANDARD_CONTENT_HEIGHT (520), shared
with McpPreferencePage's two stacked groups, so Copilot preference pages
settle at a consistent size.

Add a generic preference-pages test plan with a dialog-size regression
case.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…icrosoft#316)

microsoft#309)

microsoft#285)

Co-authored-by: Sheng Chen <sheche@microsoft.com>

)

* feat: Add profile to skip tests during UI probe execution

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

…batching (microsoft#321)

* feat: Auto-approve reads of Copilot customization files

Reading a Copilot customization file (skill, instruction, prompt, or
agent) is normal agent operation, but these reads previously triggered
a confirmation dialog — including workspace instructions caught by the
default deny rule and user-global files outside the workspace.

Recognize these reads precisely from the language server, which is the
source of truth for where the files live (including user-configured and
global locations). New copilot/customSkill|customPrompt|customInstruction|
customAgent/list requests report each file's on-disk URI; a read is
auto-approved when it exactly matches a reported file or sits inside a
reported skill folder (covering SKILL.md and its helper files). Editing
is never auto-approved — only reads.

- Add CustomizationFileService, refreshed per type on the matching
  custom*/didChange notification, exposing an immutable path snapshot.
- Hook FileOperationConfirmationHandler to auto-approve FILE_READ of a
  recognized customization file before the outside-workspace check.
- Add FileUtils.isPathWithin for skill-folder containment.

Tests: FileUtilsTests (isPathWithin), FileOperationConfirmationHandlerTests
(isCustomizationRead), and TC-010/TC-011 in the auto-approve test plan.

* rename refresh functions and merge workspaceFoldersParams

* refactor: Decouple customization file service via event bus

* rename refreshAll to refreshAllAsync

…microsoft#334)

* fix: consolidate control validation into layoutNow

requestLayout and flushPendingLayoutRequests each filtered out null/
disposed controls via a shared validControls helper before delegating
to layoutNow, duplicating the same defensive check at every call site.

layoutNow now performs its own null/disposed filtering right where it
dereferences the controls (getShell(), shell.layout(...)), so the
invariant is enforced in exactly one place instead of trusted by two
separate callers. requestLayout becomes a pure dispatcher: it no
longer pre-filters, it just forwards the raw controls to
pendingLayoutRequests or layoutNow.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* fix: repair syntax broken by autofix commit in requestLayout

The prior "Potential fix for pull request finding" commit (04e7682)
badly spliced a null-check into requestLayout, leaving a dangling
duplicate `layoutNow(changed); }` outside the method body and
dedenting the whole method. This left invalid Java that failed the
Tycho/ECJ compile in CI.

Restores correct syntax/indentation while keeping the null-guard
around Collections.addAll(pendingLayoutRequests, changed), which is
the same defensive check layoutNow already applies to its own
`controls` parameter.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* In Agent mode: Auto scroll to prompts like 'Continue'.

User must be made aware some action is needed from them to continue the
work by agent. Usually this happens when "Too many requests" or "Command
line run prompt" etc.

see https://github.com/microsoft/copilot-eclipse-feedback/issues/184

* Fix Checkstyle violations in BaseTurnWidget

- Wrap the WarnWidget construction line that exceeded 120 characters
- Fix indentation of the return statement to match the enclosing block

These were causing the CI build (Checkstyle) to fail on PR microsoft#26.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Height adjustment review fix.

We can use the target control height directly. No need to calculate by
hint.

---------

Co-authored-by: Sheng Chen <sheche@microsoft.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>