[gh-aw] Protected Files: fix: defer chat list height update to next animation frame (fixes #316501)
- Dominant language
- TypeScript
- Stars
- 193k
- Forks
- 42.4k
- PR merge metrics
- PR metrics pending
Description
> [!WARNING]
> **Protected Files**
>
> The push to pull request branch was blocked because the patch modifies protected files.
>
> **Target Pull Request:** [#316671](https://github.com/microsoft/vscode/pull/316671)
>
> **Please review the changes carefully** before pushing them to the pull request branch. These files may affect project dependencies, CI/CD pipelines, or agent behaviour.
>
>
> Protected files
>
> - `README.md`
- `package-lock.json`
- `package.json`
- `AGENTS.md`
- `CONTRIBUTING.md`
- `CODEOWNERS`
- `pnpm-lock.yaml`
- [.vscode/launch.json](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.vscode/launch.json)
- [.vscode/notebooks/endgame.github-issues](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.vscode/notebooks/endgame.github-issues)
- [.vscode/notebooks/my-endgame.github-issues](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.vscode/notebooks/my-endgame.github-issues)
- [.eslint-plugin-local/code-no-direct-gulp-import.ts](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.eslint-plugin-local/code-no-direct-gulp-import.ts)
- [.vscode/extensions/vscode-pr-pinger/package-lock.json](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.vscode/extensions/vscode-pr-pinger/package-lock.json)
- [.vscode/extensions/vscode-pr-pinger/package.json](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.vscode/extensions/vscode-pr-pinger/package.json)
- [.eslint-plugin-local/index.ts](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.eslint-plugin-local/index.ts)
- [.vscode/extensions/vscode-extras/src/npmUpToDateFeature.ts](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.vscode/extensions/vscode-extras/src/npmUpToDateFeature.ts)
- [.devcontainer/README.md](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.devcontainer/README.md)
- [.agents/skills/launch/SKILL.md](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.agents/skills/launch/SKILL.md)
- [.agents/skills/launch/scripts/launch.sh](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.agents/skills/launch/scripts/launch.sh)
- [.agents/skills/launch/scripts/monaco-paste.sh](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.agents/skills/launch/scripts/monaco-paste.sh)
- [.devcontainer/Dockerfile](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.devcontainer/Dockerfile)
- [.vscode/extensions/vscode-selfhost-test-provider/package-lock.json](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.vscode/extensions/vscode-selfhost-test-provider/package-lock.json)
- [.vscode/extensions/vscode-selfhost-test-provider/package.json](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.vscode/extensions/vscode-selfhost-test-provider/package.json)
- [.vscode/settings.json](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.vscode/settings.json)
- [.scratch-theme-test/sample.cpp](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.scratch-theme-test/sample.cpp)
- [.scratch-theme-test/sample.css](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.scratch-theme-test/sample.css)
- [.scratch-theme-test/sample.go](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.scratch-theme-test/sample.go)
- [.scratch-theme-test/sample.json](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.scratch-theme-test/sample.json)
- [.scratch-theme-test/sample.md](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.scratch-theme-test/sample.md)
- [.scratch-theme-test/sample.py](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.scratch-theme-test/sample.py)
- [.scratch-theme-test/sample.sh](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.scratch-theme-test/sample.sh)
- [.scratch-theme-test/sample.ts](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.scratch-theme-test/sample.ts)
- [.vscode/tasks.json](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.vscode/tasks.json)
- [.eslint-plugin-local/code-no-new-javascript-files.ts](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.eslint-plugin-local/code-no-new-javascript-files.ts)
- [.vscode/notebooks/api.github-issues](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.vscode/notebooks/api.github-issues)
- [.eslint-plugin-local/code-no-untyped-meta-access.ts](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.eslint-plugin-local/code-no-untyped-meta-access.ts)
- [.vscode/cglicenses.schema.json](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.vscode/cglicenses.schema.json)
- [.eslint-plugin-local/code-no-deprecated-copilot-sdk.ts](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.eslint-plugin-local/code-no-deprecated-copilot-sdk.ts)
- [.devcontainer/devcontainer-lock.json](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.devcontainer/devcontainer-lock.json)
- [.devcontainer/devcontainer.json](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.devcontainer/devcontainer.json)
- [.eslint-plugin-local/package.json](https://github.com/microsoft/vscode/blob/fix/resize-observer-loop-316501-e5237abb1e1fd22a/.eslint-plugin-local/package.json)
>
>
---
Apply the patch after review
The patch is available in the workflow run artifacts:
**Workflow Run:** [View run details and download patch artifact](https://github.com/microsoft/vscode-engineering/actions/runs/30417300124)
```sh
# Download the artifact from the workflow run
gh run download 30417300124 -n agent -D /tmp/agent-30417300124
# Apply the patch to the pull request branch
git fetch origin fix/resize-observer-loop-316501-e5237abb1e1fd22a
git checkout fix/resize-observer-loop-316501-e5237abb1e1fd22a
git am --3way /tmp/agent-30417300124/aw-microsoft-vscode-fix-resize-observer-loop-316501-e5237abb1e1fd22a.patch
git push origin fix/resize-observer-loop-316501-e5237abb1e1fd22a
```
To route changes like this to a review issue instead of blocking, configure `protected-files: fallback-to-issue` in your workflow configuration.
Contributor guide
Research direction
Review target pull request #316671 and the patch artifact from workflow run 30417300124 on branch fix/resize-observer-loop-316501-e5237abb1e1fd22a. Confirm the protected-file changes and the deferred chat list height update, then determine whether the reviewed patch should be applied to that pull request branch.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- git, github, github-actions
- Domain
- ci-cd, tooling
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100