sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: godot
Issue: Save/Loading PackedScene creates orphans
Labels: discussion,topic:core

### Tested versions

v4.2.stable.official [46dc27791]

### System information

Godot v4.2.stable - Windows 10.0.22631 - GLES3 (Compatibility) - Intel(R) Iris(R) Plus Graphics (Intel Corporation; 27.20.100.9664) - Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz (8 Threads)

### Issue description

When saving and loading using PackedScene, stray nodes are consistently created even with very simple setups.

### Steps to reproduce

1. open and run project
2. click "make pond"
3. click "spawn duck" at least once
4. click "save"
5. click "load"
6. click "print orphan", you should see that all child nodes of `duck.tscn` are printed as orphans despite also appearing in game

### Minimal reproduction project (MRP)

[PackedScene save test.zip](https://github.com/godotengine/godot/files/14895203/PackedScene.save.test.zip)
Repository: terminal
Issue: Cancel tab rename if I click elsewhere without editing the title
Labels: Issue-Feature,Help Wanted,Area-UserInterface,Product-Terminal

<!-- 
🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨

I ACKNOWLEDGE THE FOLLOWING BEFORE PROCEEDING:
1. If I delete this entire template and go my own path, the core team may close my issue without further explanation or engagement.
2. If I list multiple bugs/concerns in this one issue, the core team may close my issue without further explanation or engagement.
3. If I write an issue that has many duplicates, the core team may close my issue without further explanation or engagement (and without necessarily spending time to find the exact duplicate ID number).
4. If I leave the title incomplete when filing the issue, the core team may close my issue without further explanation or engagement.
5. If I file something completely blank in the body, the core team may close my issue without further explanation or engagement.

All good? Then proceed!
-->

# Description of the new feature/enhancement

I sometimes accidentally double-click the title of a tab that I only intended to click once.  Windows Terminal then opens the tab title for editing.  When I see that the tab title got the focus, I instinctively click the session pane to move the focus back there.  However, Windows Terminal interprets this click as meaning that I confirmed the rename.  After that, the tab title no longer updates when the shell attempts to change it.  To make it update again, I have to edit the title, delete all text, and confirm that.

I wish that Windows Terminal instead interpreted the click as meaning that I'm canceling the rename, if I have not edited the title.

# Proposed technical implementation details (optional)

Windows Terminal could either detect whether I edit the tab title at all while the edit box is open, or just compare the title to the existing one when the edit box loses the focus.

The Enter key should still confirm the rename even if the new title matches the current one.
Repository: godot
Issue: Considerable delay when changing 2D view -> editor script (with open doc)
Labels: discussion,topic:editor,needs testing,performance

### Tested versions

v4.3.dev5.official [89f70e98d]

### System information

w10 64

### Issue description

When I switch between 2d -> script editor and the script editor is showing the godot documentation, there is a considerable delay.
This delay does not occur if what is displayed in the script editor is a script.



https://github.com/godotengine/godot/assets/19652075/2deb0b31-8212-4f19-adfc-e0af431e3ca2



### Steps to reproduce

Show documentation in script editor and toggle between 2d -> script editor

### Minimal reproduction project (MRP)

...
Repository: react
Issue: [DevTools Bug] Children cannot be added or removed during a reorder operation.
Labels: Type: Bug,Status: Unconfirmed,Component: Developer Tools

### Website or app

https://redeemerllc.com

### Repro steps

when searching the products


### How often does this bug happen?

Often

### DevTools package (automated)

react-devtools-extensions

### DevTools version (automated)

5.0.2-47cf347e4

### Error message (automated)

Children cannot be added or removed during a reorder operation.

### Error call stack (automated)

```text
at chrome-extension://fmkadmapgofadopljbjfkapdkoienihi/build/main.js:1:1134857
    at A.emit (chrome-extension://fmkadmapgofadopljbjfkapdkoienihi/build/main.js:1:1101500)
    at chrome-extension://fmkadmapgofadopljbjfkapdkoienihi/build/main.js:1:1103188
    at bridgeListener (chrome-extension://fmkadmapgofadopljbjfkapdkoienihi/build/main.js:1:1509150)
```


### Error component stack (automated)

_No response_

### GitHub query string (automated)

```text
https://api.github.com/search/issues?q=Children cannot be added or removed during a reorder operation. in:title is:issue is:open is:public label:"Component: Developer Tools" repo:facebook/react
```
Repository: java-design-patterns
Issue: Publish/Subscribe pattern
Labels: epic: pattern,type: feature

Introduce the Publish/Subscribe pattern to provide a flexible event-handling system.

The Publish/Subscribe pattern is a messaging paradigm used in software architecture with several key points:

* Decoupling of publishers and subscribers: Publishers and subscribers operate independently, and there's no direct link between them. This enhances the scalability and modularity of applications.

* Event-driven communication: The pattern facilitates event-driven architectures by allowing publishers to broadcast events without concerning themselves with who receives the events.

* Dynamic subscription: Subscribers can dynamically choose to listen for specific events or messages they are interested in, often by subscribing to a particular topic or channel.

* Asynchronous processing: The pattern inherently supports asynchronous message processing, enabling efficient handling of events and improving application responsiveness.

* Scalability: By decoupling senders and receivers, the pattern can support a large number of publishers and subscribers, making it suitable for scalable systems.

* Flexibility and adaptability: New subscribers or publishers can be added to the system without significant changes to the existing components, making the system highly adaptable to evolving requirements.

Acceptance Criteria:

* The implementation must follow the project's coding standards and directory structure as outlined in the contribution guidelines.
* Comprehensive unit tests must cover the new pattern implementation, ensuring all major use cases are accounted for.
* Documentation must be provided, explaining the pattern, its use cases, and how the implementation solves a particular problem, including UML diagrams where applicable.
Repository: flutter
Issue: [image_picker ] Update `pickVideo` from gallery to use `PHPickerViewController` to avoid requested additional permissions for iOS 14+
Labels: platform-ios,p: image_picker,package,P2,team-ios,triaged-ios

### What package does this bug report belong to?

image_picker

### What target platforms are you seeing this bug on?

iOS

### Have you already upgraded your packages?

Yes

### Dependency versions

<details><summary>pubspec.lock</summary>

```lock
  image_picker:
    dependency: "direct main"
    description:
      name: image_picker
      sha256: "26222b01a0c9a2c8fe02fc90b8208bd3325da5ed1f4a2acabf75939031ac0bdd"
      url: "https://pub.dev"
    source: hosted
    version: "1.0.7"
  image_picker_ios:
    dependency: transitive
    description:
      name: image_picker_ios
      sha256: fadafce49e8569257a0cad56d24438a6fa1f0cbd7ee0af9b631f7492818a4ca3
      url: "https://pub.dev"
    source: hosted
    version: "0.8.9+1"
```

</details>


### Steps to reproduce

I have consulted the documentation of `UIImagePickerController` and `PHPickerViewController`, and I can pull up the album selector without any permission. May I ask why `pickVideo` uses `checkCameraAuthorizationWithImagePicker` and `checkPhotoAuthorizationWithImagePicker` to request permissions by default?

1. https://discussionschinese.apple.com/thread/253834114?answerId=257189479322#257189479322
2. https://developer.apple.com/documentation/photokit/selecting_photos_and_videos_in_ios

https://github.com/flutter/packages/blob/0eead7cb335048092721c59d89936a985176b20d/packages/image_picker/image_picker_ios/ios/Classes/FLTImagePickerPlugin.m#L268-L275


### Expected results

No need to request additional permissions from the user.

### Actual results

Additional permissions requested.

### Code sample

<details open><summary>Code sample</summary>

```dart
ImagePicker().pickVideo(
  source: source,
  maxDuration: const Duration(seconds: 3600),
);
```

</details>


### Screenshots or Videos

_No response_
Repository: tauri
Issue: [bug] updater creates desktop shortcut even if user selects to not create desktop shortcut in first install
Labels: type: bug,status: needs triage

### Describe the bug

Even if user select to not create desktop shortcut in first install, the updater creates desktop shortcut.

I'm using nsis installer for updater

Downstream issue: vrc-get/vrc-get#734

### Reproduction

Real-world case:

1. Download some outdated version of vrc-get-gui. https://github.com/vrc-get/vrc-get/releases/tag/gui-v0.1.0-beta.16
2. Install vrc-get-gui without creating desktop shortcut
3. Launch vrc-get-gui and upgrade vrc-get-gui with tauri updater feature
4. The desktop shortcut is created

### Expected behavior

No desktop shortcut is crated.

### Full `tauri info` output

```text
[✔] Environment
    - OS: Mac OS 14.3.1 X64
    ✔ Xcode Command Line Tools: installed
    ✔ rustc: 1.75.0 (82e1608df 2023-12-21)
    ✔ cargo: 1.75.0 (1d8b05cdd 2023-11-20)
    ✔ rustup: 1.26.0 (5af9b9484 2023-04-05)
    ✔ Rust toolchain: stable-aarch64-apple-darwin (default)
    - node: 20.10.0
    - pnpm: 8.15.4
    - yarn: 1.22.19
    - npm: 10.2.3

[-] Packages
    - tauri [RUST]: 1.6.1
    - tauri-build [RUST]: 1.5.1
    - wry [RUST]: 0.24.7
    - tao [RUST]: 0.16.8
    - tauri-cli [RUST]: 2.0.0-beta.11
    - @tauri-apps/api [NPM]: 1.5.3
    - @tauri-apps/cli [NPM]: 1.5.11

[-] App
    - build-type: bundle
    - CSP: unset
    - distDir: out
    - devPath: http://localhost:3000/
    - framework: React (Next.js)
    - bundler: Webpack
```


### Stack trace

_No response_

### Additional context

_No response_
Repository: terminal
Issue: window explorer (and terminal) hangs when I edit the address bar in a file picker
Labels: Issue-Bug,Product-Terminal,Area-SettingsUI

### Windows Terminal version

1.19.10573.0

### Windows build number

 10.0.19045

### Other Software

Windows explore
![image](https://github.com/microsoft/terminal/assets/36665957/3dadd74a-3519-4782-bab8-fc16d4912582)


### Steps to reproduce

In the icon option for editing a single profile, after using the browse option and opening window explorer, it is not possible to enter the path to the file directly in the address bar, it is not possible to enter any characters, which causes the entire Windows terminal to go down, and the only way to close the program is to close it through task management.
please look the gif animation

### Expected Behavior

![image](https://github.com/microsoft/terminal/assets/36665957/d759f07b-c5e5-4f75-817c-94a7bfee3d49)


### Actual Behavior

software is dead
Repository: next.js
Issue: Set multiple cookies on API routes (Pages Router)
Labels: bug,Runtime

### Link to the code that reproduces this issue

https://github.com/nicolascavallin/next-pages-firebase-auth-boilerplate/blob/main/src/pages/api/auth/session.ts

### To Reproduce

## In API handler (Pages Router)

- Set two cookies with `res.setHeader("Set-Cookie": [cookie1, cookie2])`

- Then: if you are running local (`pnpm dev` and `pnpm build + pnpm start`), both cookies will be settled. 

- Deploy to Vercel: call the same API, only first cookie will be settled.

If changes the order of cookies, then again, first will be settled (in Vercel).

## In API route (App Router)

- Set two cookies with `cookies()`

- Then: both cookies are settled in Vercel and in Local.

### Current vs. Expected behavior

I expect to settled both cookies on Pages Router API.

## Current (Vercel)

<img width="431" alt="Screenshot 2024-04-07 at 12 24 30" src="https://github.com/vercel/next.js/assets/22961489/0776b25f-169b-44cb-98a4-b56118659204">

## Current (Local with HTTPS)

<img width="472" alt="Screenshot 2024-04-07 at 12 25 05" src="https://github.com/vercel/next.js/assets/22961489/0b207716-8a88-48ba-896e-698a0fe49242">


### Provide environment information

```bash
Operating System:
  Platform: darwin
  Arch: arm64
  Version: Darwin Kernel Version 23.4.0: Fri Mar 15 00:12:49 PDT 2024; root:xnu-10063.101.17~1/RELEASE_ARM64_T6020
Binaries:
  Node: 20.10.0
  npm: 10.2.3
  Yarn: 1.22.19
  pnpm: 8.12.1
Relevant Packages:
  next: 14.1.4
  eslint-config-next: 14.1.4
  react: 18.2.0
  react-dom: 18.2.0
  typescript: 5.4.4
Next.js Config:
  output: N/A
```


### Which area(s) are affected? (Select all that apply)

Not sure, Middleware / Edge (API routes, runtime)

### Which stage(s) are affected? (Select all that apply)

Vercel (Deployed)

### Additional context

I suspect a lot of things and play with all properties: httpOnly, secure, sameSite, priority.

Also, suspecting a "big size" of data, I just try to set simple cookies like: `foo1=bar1` and `foo2=bar2`. 

Always getting the same result, only first cookie settled.

Also I tried

```javascript
res.setHeader("Set-Cookie": cookie1)
res.setHeader("Set-Cookie": cookie2)
```

```javascript
res.setHeader("Set-Cookie": `${cookie1}; ${cookie2}`)
```

And of course

```javascript
res.setHeader("Set-Cookie": [cookie1, cookie2])
```

And nothing.

I tried to get help from `cookies-next` library... nothing.

I didn't found any solution and any similar issue, so I'm opening this one.

Thanks in advance.
Repository: next.js
Issue: HMR broken for runtime edge when using "use client" directive
Labels: bug,Runtime

### Link to the code that reproduces this issue

https://github.com/zn4rk/nextjs-edge-runtime-hmr

### To Reproduce

1. Start the application in development (next dev).
2. Edit anything in app/page.tsx that would cause HMR to take effect.
3. Watch the browser reload.

### Current vs. Expected behavior

The current behavior is that the page reloads instead of HMR taking effect:

 ⚠ Fast Refresh had to perform a full reload. Read more: https://nextjs.org/docs/messages/fast-refresh-reload

Expected behavior is that HMR would work.

By removing the "use client"-directive, it starts to work again.

### Provide environment information

```bash
Operating System:
  Platform: darwin
  Arch: arm64
  Version: Darwin Kernel Version 21.6.0: Thu Jun  8 23:56:13 PDT 2023; root:xnu-8020.240.18.701.6~1/RELEASE_ARM64_T6000
  Available memory (MB): 65536
  Available CPU cores: 10
Binaries:
  Node: 20.9.0
  npm: 10.1.0
  Yarn: 1.22.18
  pnpm: 8.15.4
Relevant Packages:
  next: 14.2.0-canary.61 // Latest available version is detected (14.2.0-canary.61).
  eslint-config-next: N/A
  react: 18.2.0
  react-dom: 18.2.0
  typescript: 5.1.3
Next.js Config:
  output: N/A
```


### Which area(s) are affected? (Select all that apply)

App Router, Middleware / Edge (API routes, runtime)

### Which stage(s) are affected? (Select all that apply)

next dev (local)

### Additional context

I found many open issues that are related to HMR, but none explicitly covers this case.

https://github.com/vercel/next.js/issues/51162
https://github.com/vercel/next.js/issues/54313
Repository: TypeScript
Issue: Function lacks ending return statement and return type does not include 'undefined'
Labels: Suggestion,Awaiting More Feedback

### 🔎 Search Terms

switch default

### 🕗 Version & Regression Information

The tsc complains that the the function return undefined if a add a default case, actually the default case will never be executed, just only for exhausive check.

### ⏯ Playground Link

https://www.typescriptlang.org/play?jsx=0&ts=5.4.4#code/C4TwDgpgBAyglgL2gXigcgDYEMBOBzCNKAH3QDsB7HAWywyNLQGdaN6T1q4y40AoAMYUyTYFCaIUUAN4BfKFiaxJg4aKgAjDBQEBrKKgA8AJQB8ACgBmALijmAlAdNRjj5M+l8o3qDgjAAVxwyKEsHPlk+VRExASYlVC0dXXMHWzIA6g0IHCcZLx8mAHc4YAEACzsJJEdPH3qoAUVoNEoaOjRbOoaev0DgqABGAp7Inu8mphbsfEIukfHvPqCQgCYF+rHxyZauHk78xfrlgYBaQagNny2enfQWOnp5o58TkIAGK+8bhoATCEsWACGGAzxejTUYjI6QgADccgZxCpFj9IrJ7EA

### 💻 Code

```ts
type Size = 'large' | 'normal' | 'small' | 'mini'
const size = {} as Size
const block = <R>(f: () => R) => {
    return f()
}

const css = block((): number => {
    switch (size) {
        case 'normal': {
            return 1
        }
        case 'large': {
            return 2
        }
        case 'mini': {
            return -1 
        }
        case 'small': {
            return 0
        }
        default: {
            const n: never = size
        }
    }
})
```


### 🙁 Actual behavior

The tsc think that the function of css block returns number | undefined.

### 🙂 Expected behavior

The tsc should know that the default case is never returned, and the function of css block should returns number

### Additional information about the issue

_No response_
Repository: godot
Issue: ResourceLoader doesn't seem to cache loaded assets even with CACHE_MODE_REUSE
Labels: topic:core,documentation

### Tested versions

- Reproducible in 4.1-stable, 4.2.1-stable

### System information

Godot v4.1.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 2070 SUPER (NVIDIA; 31.0.15.3623) - AMD Ryzen 7 3700X 8-Core Processor (16 Threads)

### Issue description

### The issue
`ResourceLoader.load` doesn't seem to cache the asset within ResourceLoader itself even with CACHE_MODE_REUSE.

The code that prints `false` in `ResourceLoader.has_cached`:
```gdscript
func _non_cached_load():
    ResourceLoader.load("res://TestResource.tres", "", ResourceLoader.CACHE_MODE_REUSE)
    print(ResourceLoader.has_cached("res://TestResource.tres"))
```

The code below prints `true` in `ResourceLoader.has_cached`:
```gdscript
func _stack_cached_load():
    var x = ResourceLoader.load("res://TestResource.tres", "", ResourceLoader.CACHE_MODE_REUSE)
    print(ResourceLoader.has_cached("res://TestResource.tres"))
```

`_stack_cached_load()` works because `x` is still within scope in the function. The asset will fall out of scope as soon as it code execution exits the function.

### Expected behavior
`_non_cached_load()` should return `true`.


### Steps to reproduce

1. Create a resource in the project.
2. Create a function similar to what is described in `_non_cached_load`.
3. The return is false.

### Minimal reproduction project (MRP)

[MRPLoadCacheBug.zip](https://github.com/godotengine/godot/files/14897218/MRPLoadCacheBug.zip)
Repository: godot
Issue: There is a small error in the root motion calculation in some cases
Labels: bug,topic:animation

### Tested versions

4.3.dev (f8bae10be6a5d33acf8409b915aba9dd6493dcf6)

### System information

Any

### Issue description

Probably the root motion is incorrectly processed in one frame at the beginning or end of animation.

**Errors occur in the following cases:**

| | AnimationTree | AnimationPlayer |
| -- | -- | -- |
| Idle | Error (ahead) | Error (delay) |
| Physics | No error | Error (delay) |

The AnimationTree / Idle combination was correct as of 4.2.stable, but regression was caused by #87171.
The root motion for AnimationPlayer was implemented in 4.2, but it already has been got error as of 4.2.stable.

### Steps to reproduce

play "Left" animation in MRP. The animation is expected to rotate 90 degrees and stop, but error occurs after the animation is played.

![image](https://github.com/godotengine/godot/assets/61938263/7bc0571b-5c5d-4a69-886b-4bcfee5fb1ff)

### Minimal reproduction project (MRP)

[rootmotion.zip](https://github.com/godotengine/godot/files/14897421/rootmotion.zip)
Repository: PowerToys
Issue: Latest update changes on-desk icons
Labels: Issue-Bug,Area-Setup/Install,Needs-Triage,Needs-Team-Response

### Microsoft PowerToys version

0.80.0

### Installation method

PowerToys auto-update

### Running as admin

Yes

### Area(s) with issue?

General

### Steps to reproduce

After last update my on-screen icons mostly changed to the default image

### ✔️ Expected Behavior

No change in on screen icons

### ❌ Actual Behavior

Icon's changed see screenshot
![Screenshot 2024-04-07 150141](https://github.com/microsoft/PowerToys/assets/68851732/e1a5c8d9-9223-430b-a8cd-e0ea3941287c)
[PowerToysReport_2024-04-07-14-54-27.zip](https://github.com/microsoft/PowerToys/files/14897478/PowerToysReport_2024-04-07-14-54-27.zip)


### Other Software

Nothing that should effect this
Repository: ui
Issue: [bug]: Init creates `@` folder instead of alias for /components and /lib/utils.ts
Labels: bug

### Describe the bug

When using the init command leaving everything with defaults the output of the aliases is a folder
@/components/ui
@/lib/utils.ts
   
   
Instead we should get
~/components/ui
~/lib/utils.ts


Think the issue here: https://github.com/shadcn-ui/ui/blob/7590fb76364c46b72653623b6ee4a96f0ad0bca8/packages/cli/src/commands/init.ts#L327

when `@/components/ui` and `@/lib` are provided it creates the `@/` folder.




### Affected component/components

ALL

### How to reproduce

1. `pnpm dlx shadcn-ui@latest init`
2. All defaults
3. Created folders @/components/ui & @lib/utils

<img width="742" alt="Screenshot 2024-04-07 at 11 52 30 PM" src="https://github.com/shadcn-ui/ui/assets/10654469/f73c8692-e616-4c16-a104-f7a5f6e094b3">
<img width="711" alt="Screenshot 2024-04-07 at 11 52 48 PM" src="https://github.com/shadcn-ui/ui/assets/10654469/912e6ad1-3db4-4c2e-8eb7-2e68267c9114">
<img width="331" alt="Screenshot 2024-04-07 at 11 55 57 PM" src="https://github.com/shadcn-ui/ui/assets/10654469/60ffc625-97d5-47e0-9cf8-4f14583ee4e5">


### Codesandbox/StackBlitz link

_No response_

### Logs

_No response_

### System Info

```bash
macOS: Sonoma 14.4.1 (23E224)
```


### Before submitting

- [X] I've made research efforts and searched the documentation
- [X] I've searched for existing issues