sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: TypeScript
Issue: Codefix `convertTypedefToType` creates types in a wrong place
Labels: Bug,Help Wanted,Domain: Refactorings,Rescheduled,Effort: Casual

Currently "typedef to type conversion" doesn't care about the place where type is generated, resulting broken code if the type if generated inside the class / interface etc.

It's better to generate class in closest SourceFile root, namespace root or even function body in case the comment containing @typedef is placed at any other place of the code.

### 🔎 Search Terms

Searched for typedef issues reported after [March 1st](https://github.com/microsoft/TypeScript/pull/51430/files)

### 🕗 Version & Regression Information

- This changed between versions 5.1.x and latest


### ⏯ Playground Link

[Link to play](https://www.typescriptlang.org/play?ts=5.1.6&ssl=1&ssc=8&pln=1&pc=1&filetype=js#code/KYDwDg9gTgLgBAYwDYEMDOa4FEQoLZhLBwDeAUHHAPQBUNFlNcAMgJZrwQBmcrMweTDAhwARsSjAAdgBNgkmbylwYAC2KiUURKq0wGcJgAEYATzDA5PEiUQQArlJgAuOFPt5x2gL7e4AYQcneQNjSRh7KClMEkDHfihvUKoGAHNgeDQIPAzVVilUgAoASlIDOHDI5VsEIJc4AAY4bwBuBiTvIA)

### 💻 Code

```
class Example {
  /**
   * List of items to be rendered in the bar chart
   * @typedef {{ count: number }} Counter
   * @returns {Counter}
   */
  get something() {
    return { count: 0 };
  }
}
```

### 🙁 Actual behavior
```
class Example {
  type Counter={ count: number; };     // <- bug is here
  get something() {
    return { count: 0 };
  }
}
```

### 🙂 Expected behavior
```
type Counter={ count: number; };

class Example {
  /**
   * List of items to be rendered in the bar chart
   * @returns {Counter}
   */
  get something() {
    return { count: 0 };
  }
}
```
Repository: PowerToys
Issue: Right click menu in system tray doesn't follow dark mode 
Labels: Issue-Bug,Product-Settings,Priority-1,Status-Reproducible

### Microsoft PowerToys version

0.72.0

### Installation method

WinGet

### Running as admin

No

### Area(s) with issue?

System tray interaction

### Steps to reproduce

right click on the icon in the system tray of PT or any PT's tool

### ✔️ Expected Behavior

The menu is displayed in dark mode if it's dark mode is on, like: 
<img width="205" alt="image" src="https://github.com/microsoft/PowerToys/assets/92474350/b0d1a521-7f26-46ab-9ebf-e6dbd3e67ab5">


### ❌ Actual Behavior

The menu is in white mode anyway
<img width="290" alt="image" src="https://github.com/microsoft/PowerToys/assets/92474350/8a2f4e50-ac28-4160-acf7-db28f56e6de2">
(not just with the main icon)
<img width="274" alt="image" src="https://github.com/microsoft/PowerToys/assets/92474350/52cc9d3a-e57d-4bfc-8271-0f4028233ed4">


### Other Software

_No response_
Repository: kubernetes
Issue: authn: consider expanding oidc API to allow warnings to be returned to the user
Labels: sig/auth,needs-triage

We would need to think through how this should work in the generic sense as well, as well as how it would interact with delegated flows.  Would the warnings be set via the context or through a dedicated field on the authentication response?  For token review and SAR, it would probably make sense for this to be part of the in-band API response to match webhook admission.

Conversation I had with Jordan related to this:

> actual request to kube-apiserver → tokenreview → warnings get copied to header for requesting user
> tokenreview request to kube-apiserver → delegated tokenreview → warnings get copied to status of tokenreview returned?

/sig auth

xref: #90073

cc @aramase
Repository: godot
Issue: [Control/2D] Zooming in viewport makes objects "bleed" outside their desired size (Pixel Perfect, Anti-Aliasing issue)
Labels: documentation,topic:gui

### Godot version

v4.1.1.stable.official [bd6af8e0e]

### System information

Linux (Fedora 38)

### Issue description

https://github.com/godotengine/godot/assets/748770/955254c9-5fae-4c75-9c8a-420045d866e3



### Steps to reproduce

1. Create a godot button
2. Create a godot vseparator and drag near the edge to compare a hard edge vs soft edge
3. Zoom in to see bleed outside the control edges and border
4. Try changing texture filtering options (doesn't resolve issue)

### Minimal reproduction project

[TestHardEdge.zip](https://github.com/godotengine/godot/files/12294709/TestHardEdge.zip)
Repository: Web-Dev-For-Beginners
Issue: There should be command guidance to set up the project locally
Labels: no-issue-activity

**Is your feature request related to a problem? Please describe.**
In the guidance to setup the project locally or offline ,there is not proper guidance provided how to set up .There should be available proper commands to make the contributor easier to set up the project 

**Describe the solution you'd like**
There should provided some commands which helps in setting up the project offline or locally


**Additional context**

![Screenshot (182)](https://github.com/microsoft/Web-Dev-For-Beginners/assets/138685168/47a28fe9-f903-4eda-ac63-d2ff5ea6193a)
Repository: vscode
Issue: [Accessibility] Add a setting for users to choose whether to hear partial or the complete token in inline suggestion alert
Labels: feature-request,accessibility,inline-completions


Type: <b>Feature Request</b>

When inline suggestion is triggered, SR users only hear the  partial token. For example, if you type "lib" and triggered the inline suggestion, you will hear "ibrary()" for "library()" because "lib" is already typed by users.

Personally, I like the default behavior, but announcing the complete token might be insturmental for novice users. I suggest adding an option for users to choose this behavior in the setting.

VS Code version: Code - Insiders 1.82.0-insider (85daa35afb86202badbee79532537bf226961f25, 2023-08-08T07:41:53.755Z)
OS version: Windows_NT x64 10.0.22621
Modes:


<!-- generated by issue reporter -->
Repository: neovim
Issue: Delete/change single character in forced blockwise motion is not consistent
Labels: bug-vim,needs:vim-patch,normal-mode

### Problem

Blockwise forced motion doesn't work consistently for single character during delete/change.

### Steps to reproduce

1. `nvim --clean`
2. Add line `ab`
3. Put cursor on `a`.
4. Create a mark on it with `mx`.
5. Type ``d<C-v>`x``.

Note: the issue is also reproducible with change operator `c`.

### Expected behavior

Character `a` is deleted. Same as after `<C-v>`xd` (delete visual selection).

This is not entirely predictable for these reasons:
- Yanking in this situation (``y<C-v>`x``) does work.
- Charwise forced mode (``dv`x``) does work.
- If mark is over `b`, then typing ``d<C-v>`x`` will delete both characters leaving no way to actually delete single character in blockwise forced mode.

### Neovim version (nvim -v)

NVIM v0.10.0-dev-772+g0211f889b

### Vim (not Nvim) behaves the same?

Yes (9.1.1676)

### Operating system/version

EndeavourOS Linux x86_64 (6.4.8-arch1-1)

### Terminal name/version

st-0.9

### $TERM environment variable

st-256color

### Installation

appimage
Repository: flutter
Issue: Warning in log when using google_maps_flutter v2.4.0: `Google Maps JavaScript API has been loaded directly without a callback.`
Labels: d: api docs,p: maps,platform-web,package,P2,c: tech-debt,team-web,triaged-web

Warning in log when using `google_maps_flutter v2.4.0.`

`Google Maps JavaScript API has been loaded directly without a callback. This is not supported and can lead to race conditions and suboptimal performance. For supported loading patterns please see https://goo.gle/js-api-loading`

Isn't `google_maps_flutter` v2.4.0 loading the map correctly internally ?
Repository: flutter
Issue: build/config/linux/pkg-config.py should not append sysroot paths
Labels: engine,c: proposal,P3,team-engine,triaged-engine

### Is there an existing issue for this?

- [X] I have searched the [existing issues](https://github.com/flutter/flutter/issues)
- [X] I have read the [guide to filing a bug](https://flutter.dev/docs/resources/bug-reports)

### Steps to reproduce

`git clone https://gitlab.com/aduskett/buildroot-flutter.git`
`cd buildroot-flutter`
`make flutter_aarch64_defconfig`
`rm package/flutter-engine/0004-pkg-config.py-do-not-prepend-sysroot-path.patch`
`make -Otarget -j$(nproc) flutter-engine-depends`
`make flutter-engine`

### Actual results

Buildroot uses the standard PKG_CONFIG_SYSROOT_DIR variable, so what the  pkg-config.py script is doing doesn't make sense. There is no need to prepend the sysroot, and flutter should let pkg-config use the  PKG_CONFIG_SYSROOT_DIR variable.

Without the patch, the pkg-config.py script double prepends the sysroot path.
IE: output/host/.../sysroot/output/host/.../sysroot

### Logs

<details>
<summary>Logs</summary>

```console

<!-- Paste your logs here -->

```

</details>


### Flutter Doctor output

<details>
<summary>Doctor output</summary>

```console

<!-- Paste your output here -->

```

</details>
Repository: godot
Issue: Inconsistent default values in TileMap.set_cell() and TileMapPattern.set_cell()
Labels: discussion,usability,documentation,topic:2d

### Godot version

v4.1.1.stable.official [bd6af8e0e]

### System information

Godot v4.1.1.stable - Windows 10.0.19045 - Vulkan (Compatibility) - Radeon RX 570 Series (Advanced Micro Devices, Inc.; 31.0.21001.45002) - Intel(R) Core(TM) i3-8100 CPU @ 3.60GHz (4 Threads)

### Issue description

TileMap.set_cell() and TileMapPattern.set_cell() have different default values for the alternative_tile parameter. TileMap.set_cell() defaults to 0, allowing a tile to be placed at the given coordinates if no value is provided, while TileMapPattern.set_cell() defaults to -1, which instead erases the cell at the given coordinates.
My expectation was that the behavior of these functions would be consistent, given their similarities and that the documentation for TileMapPattern.set_cell() tells users to refer to TileMap.set_cell(). I am not sure whether the default value for both functions should be 0 or -1, and ask that someone more familiar with TileSets make the decision.

### Steps to reproduce

1. Create a TileMap.
2. Create a TileSetAtlasSource, add a texture to it, and create a tile.
3. Add the TileSetAtlasSource to the TileMap.
4. Create a TileMapPattern.
5. Use TileMapPattern.set_cell(), using the source_id of the TileSetAtlasSource and the atlas_coords of the created tile. Do not specify an alternative tile.
6. In the TileMap, set a cell using TileMap.set_cell(). Do not specify an alternative tile.
7. In the TileMap. set the TileMapPattern at different coordinates using TileMap.set_pattern().
8. The tile placed using TileMap.set_cell() will appear normally, while the tile(s) placed using TileMap.set_pattern() will not appear.

### Minimal reproduction project

N/A
Repository: flutter
Issue: [image_picker][iOS] Selecting an AVIF file on iOS causes indefinite loader and no response / crash
Labels: platform-ios,p: image_picker,package,e: OS-version specific,has reproducible steps,P2,found in release: 3.10,team-ios,triaged-ios,found in release: 3.13

### Is there an existing issue for this?

- [X] I have searched the [existing issues](https://github.com/flutter/flutter/issues)
- [X] I have read the [guide to filing a bug](https://flutter.dev/docs/resources/bug-reports)

### Steps to reproduce

Open up either the multi image selection or single image selection. Select an AVIF file (that can be sent to the iOS device). ImagePicker will show indefinite loader and return no response. No choice but to stop app and rebuild.

### Expected results

I would probably expect some sort of error message. But the ideal expected result is some feedback saying that AVIF files cannot be chosen or if those images in the Photo app are disabled entirely for selection.

### Actual results

Indefinite loader and crash

### Code sample

final picker = ImagePicker();
var pickedFiles = await picker.pickMultiImage();

return pickedFiles;

### Screenshots or Video

<details>
<summary>Screenshots / Video demonstration</summary>

[Upload media here]

</details>


### Logs

_No response_

### Flutter Doctor output

[✓] Flutter (Channel stable, 3.7.12, on macOS 13.1 22C65 darwin-arm64 (Rosetta), locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.2)
[✓] VS Code (version 1.81.0)
[✓] Connected device (4 available)
    ! Error: (null) needs to connect to determine its availability. Check the connection between the device and its companion iPhone, and the connection between the iPhone and
      Xcode. Both devices may also need to be restarted and unlocked. (code 1)
    ! Error: KJ’s iPhone is busy: Fetching debug symbols for KJ’s iPhone. Xcode will continue when KJ’s iPhone is finished. (code -10)
[✓] HTTP Host Availability

• No issues found!
Repository: go
Issue: x/net/http2: singleUse ClientConn with AllowHTTP transport is always unusable 
Labels: NeedsInvestigation

### What version of Go are you using (`go version`)?

<pre>
$ go version 
go version go1.20.2 darwin/arm64
</pre>

### Does this issue reproduce with the latest release?
Yes


### What operating system and processor architecture are you using (`go env`)?

<details><summary><code>go env</code> Output</summary><br><pre>
$ go env
GO111MODULE="on"
GOARCH="arm64"
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPROXY="direct"
GOROOT="/opt/homebrew/Cellar/go/1.20.2/libexec"
GOSUMDB="sum.golang.org"
GOTOOLDIR="/opt/homebrew/Cellar/go/1.20.2/libexec/pkg/tool/darwin_arm64"
GOVERSION="go1.20.2"
GCCGO="gccgo"
AR="ar"
CC="cc"
CXX="c++"
CGO_ENABLED="1"
CGO_CFLAGS="-O2 -g"
CGO_CXXFLAGS="-O2 -g"
CGO_FFLAGS="-O2 -g"
CGO_LDFLAGS="-O2 -g"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/g4/47vwdw1s65929qw7smyq1vlswhhzgj/T/go-build4150723989=/tmp/go-build -gno-record-gcc-switches -fno-common"
</pre></details>

### What did you do?

Create an HTTP2 transport with AllowHTTP = true, and try to send a request using it with req.Close = true. This will hang until the internal retry mechanism in the HTTP2 roundtrip ends after exponential backoff.

https://go.dev/play/p/KhmQcQB0PPD

If you remove req.Close = true, in my contrived example you get unexpected EOF, but the point is no timeout, because its not retrying aimlessly.

### What did you expect to see?
I would expect the transport to understand that AllowHTTP=true setting nextStreamID to 3 in the newClientConn() function does not mean it should be considered "not usable" before it ever makes a request.


### What did you see instead?
```
2009/11/10 23:00:00 Starting server...
2009/11/10 23:00:00 Sending request...
<time passes because of retry backoff>
http2: client conn not usable
```


The issue is because of [this line](https://github.com/golang/net/blob/master/http2/transport.go#L790) that sets nextStreamID to 3 before a request is ever made, and then [this line](https://github.com/golang/net/blob/c8c0290b421c479315f66c7b68b617ef6e73c668/http2/transport.go#L963) that interprets a nextStreamID higher than 1 as having been "used".

I'm not sure my use case for this is exactly "up to standard" but it seems like the behavior is unintentional - the connection should be usable once even if AllowHTTP was set on its creation, and at least it shouldn't retry exponentially when there's no way it could get a different outcome in the next iteration.

Thanks for reading!
Repository: vscode
Issue: Suggestion: make `executeSelectionRangeProvider` respect settings for `editor.smartSelect`
Labels: help wanted,feature-request,smart-select

<!-- ⚠️⚠️ Do Not Delete This! bug_report_template ⚠️⚠️ -->
<!-- Please read our Rules of Conduct: https://opensource.microsoft.com/codeofconduct/ -->
<!-- 🕮 Read our guide about submitting issues: https://github.com/microsoft/vscode/wiki/Submitting-Bugs-and-Suggestions -->
<!-- 🔎 Search existing issues to avoid creating duplicates. -->
<!-- 🧪 Test using the latest Insiders build to see if your issue has already been fixed: https://code.visualstudio.com/insiders/ -->
<!-- 💡 Instead of creating your report here, use 'Report Issue' from the 'Help' menu in VS Code to pre-fill useful information. -->
<!-- 🔧 Launch with `code --disable-extensions` to check. -->
Does this issue occur when all extensions are disabled?: Yes/No

<!-- 🪓 If you answered No above, use 'Help: Start Extension Bisect' from Command Palette to try to identify the cause. -->
<!-- 📣 Issues caused by an extension need to be reported directly to the extension publisher. The 'Help > Report Issue' dialog can assist with this. -->
- VS Code Version: 1.81.0
- OS Version: macOS latest

VS Code exposes a command called `executeSelectionRangeProvider`. This calls `provideSelectionRanges` with `{ selectLeadingAndTrailingWhitespace: true, selectSubwords: true }`:

https://github.com/microsoft/vscode/blob/42b4bf06704b84b40177f62405497a557a8fd73d/src/vs/editor/contrib/smartSelect/browser/smartSelect.ts#L308-L317

I am wondering whether VS Code should respect the smart select settings exposed under `editor.smartSelect` (e.g. `editor.smartSelect.selectLeadingAndTrailingWhitespace`), instead of passing `true` all of the time.

For context, I'm trying to create an extension that exposes the built-in "smart select" feature through a "quick picker". I've mostly got this working, except it doesn't respect the user's settings for smart select:

https://github.com/microsoft/vscode/assets/921609/52c59aea-9eb2-4439-9520-663daa4513d3

Code: https://github.com/OliverJAsh/vscode-smart-select-picker/blob/8158ebc258649c06a76ba9ec255a0ec315ba6863/src/extension.ts
Repository: flutter
Issue: [flutter_tools] RPCError "Unexpected DWDS error for resume: WipError -32000"
Labels: P2,team-tool,triaged-tool

on beta 3.13.0-0.3.pre

```
RPCError: resume: (-32603) Unexpected DWDS error for resume: WipError -32000 Can only perform operation while paused.content_copy
at new _OutstandingRequest(vm_service.dart:1928)
at VmService._call(vm_service.dart:2493)
at VmService.resume(vm_service.dart:2368)
at IsolateManager.resumeThread(isolate_manager.dart:287)
at IsolateManager._handlePause(isolate_manager.dart:498)
at IsolateManager.handleEvent(isolate_manager.dart:177)
at <asynchronous gap>(async)
at DartDebugAdapter._configureExistingIsolates.<anonymous closure>(dart.dart:757)
at <asynchronous gap>(async)
at Future.wait.<anonymous closure>(future.dart:525)
at <asynchronous gap>(async)
at DartDebugAdapter._configureExistingIsolates(dart.dart:742)
at <asynchronous gap>(async)
at DartDebugAdapter._withErrorHandling(dart.dart:2424)
at <asynchronous gap>(async)
at DartDebugAdapter.connectDebugger(dart.dart:690)
at <asynchronous gap>(async)
at FlutterDebugAdapter._connectDebugger(flutter_adapter.dart:383)
at <asynchronous gap>(async)
at FlutterDebugAdapter._handleDebugPort(flutter_adapter.dart:464)
at <asynchronous gap>(async)
```

Related fix (that should already be in this flutter branch) was https://github.com/dart-lang/webdev/pull/2134/files.

@DanTup I think the action item here would be for the dart debug adapter to handle this error. Does that make sense?
Repository: TypeScript
Issue: TypeScript does not detect a symlinked directory getting re-created
Labels: Possible Improvement

<!-- ⚠️⚠️ Do Not Delete This! bug_report_template ⚠️⚠️ -->
<!-- Please read our Rules of Conduct: https://opensource.microsoft.com/codeofconduct/ -->
<!-- 🕮 Read our guide about submitting issues: https://github.com/microsoft/vscode/wiki/Submitting-Bugs-and-Suggestions -->
<!-- 🔎 Search existing issues to avoid creating duplicates. -->
<!-- 🧪 Test using the latest Insiders build to see if your issue has already been fixed: https://code.visualstudio.com/insiders/ -->
<!-- 💡 Instead of creating your report here, use 'Report Issue' from the 'Help' menu in VS Code to pre-fill useful information. -->
<!-- 🔧 Launch with `code --disable-extensions` to check. -->
Does this issue occur when all extensions are disabled?: Yes

<!-- 🪓 If you answered No above, use 'Help: Start Extension Bisect' from Command Palette to try to identify the cause. -->
<!-- 📣 Issues caused by an extension need to be reported directly to the extension publisher. The 'Help > Report Issue' dialog can assist with this. -->
- VS Code Version: 1.80.2
- OS Version: macOS 13.4.1

Steps to Reproduce:

1. Run the following:
   ```
   mkdir -p scratch/src
   mkdir scratch2
   dst=$(cd scratch2 && pwd)
   ln -s $dst scratch/src/symlink
   echo "import { Foo } from './symlink/index'" >scratch/src/test.ts
   echo "export const Foo = 1;" >scratch2/index.ts
   code --disable-extensions scratch
   ```
   Open `test.ts`. It should have no errors.
3.  Run `rm -rf scratch2`. After vscode detects the change, it should display a red squiggle under `./symlink/index` with the error "Cannot find module ...".
4. Run `mkdir scratch2 && echo "export const Foo = 1;" >scratch2/index.ts`

The expected behavior at this point is that vscode recognizes that the target directory of the symlink once again exists, and removes the red squiggle.

The actual behavior is that the squiggle does not go away. You must close and reopen the project to get vscode to recognize that the code is valid.
Repository: flutter
Issue: [Impeller] Unify instances of Font in `impeller::GlyphAtlas`
Labels: c: new feature,engine,c: performance,P2,e: impeller,team-engine,triaged-engine

Right now a `impeller::GlyphAtlas` retains a copy of a `impeller::Font` for each `FontGlyphPair` it contains.  This results in the destructor for Fonts showing up in profiling results, about 1.5% of the raster thread time when scrolling around on the Flutter Gallery main page.

The following change would be ideal.  **It isn't a huge savings** though and is a bit involved to implement because of the usage of `FontGlyphPair` as a key to the unordered_map.

```c++
// Like FontGlyphPair but holds a reference to the Font instead of a copy.
struct FontRefGlyphPair {
  const Font& font;
  Glyph glyph;
  Scalar scale;
};

class GlyphAtlas {
  private:
    std::set<Font> fonts_;
    std::unordered_map<FontRefGlyphPair, Rect> positions_;
};
```

## profile
This is from running Flutter Gallery, expanding "Material" and just scrolling around:

<img width="808" alt="Screenshot 2023-08-08 at 2 45 23 PM" src="https://github.com/flutter/flutter/assets/30870216/b1412b06-5e94-46c7-909e-29f95d90225d">
Repository: storybook
Issue: [Bug]: I use Storybook Composition but I still see the text "Oh no! Your Storybook is empty"
Labels: bug,needs triage

### Describe the bug

I have a Storybook Host which is empty but contains several separate storybooks. And every time I open it, I see this page:
![image](https://github.com/storybookjs/storybook/assets/17424528/772b1144-3c1b-4cf3-ab8c-a5022d3ccc68)

my config doesn't even contain history, only refs:
```
const config: StorybookConfig = {
    stories: [],
    refs: {
        'lookup-accounts': {
            title: 'Exteran Storybook 1',
            url: 'http://localhost:6006',
        },
        'partner-portal': {
            title: 'Exteran Storybook 2',
            url: 'http://localhost:6007',
        },
    },
}
```

it seems logical in this case not to show the error but just open the first external history or at least show a blank page

### To Reproduce

_No response_

### System

_No response_

### Additional context

_No response_