PrimeIntellect/SWE-rebench-V2-Filtered-Easy-Verified huggingface dataset:
Repository: ember-cli/eslint-plugin-ember (js)
Issue: running --fix w/ ember/use-ember-get-and-set messes up my code
<img width="636" alt="screen shot 2017-08-14 at 4 12 50 pm" src="https://user-images.githubusercontent.com/643505/29295845-8d63733a-810b-11e7-9c62-ad8b7448c4e9.png">
<img width="630" alt="screen shot 2017-08-14 at 4 13 33 pm" src="https://user-images.githubusercontent.com/643505/29295846-8d74b8a2-810b-11e7-9175-7f31321c30eb.png">

seems like `Ember, Ember,`s keep getting added every time i run `--fix`
Pull request description: Don't report on Ember method calls in use-ember-get-and-set
This rule should not report on Ember method calls. As a result, this
fixes #130 by not trying to repeatedly fix code on successive runs of
`eslint --fix`.

Patch:
diff --git a/lib/rules/use-ember-get-and-set.js b/lib/rules/use-ember-get-and-set.js
index 73db0f49..10fb48f1 100644
--- a/lib/rules/use-ember-get-and-set.js
+++ b/lib/rules/use-ember-get-and-set.js
@@ -117,6 +117,10 @@ module.exports = {
         ) {
           return;
         }
+        // Skip calls made on Ember methods
+        if (isIdentifier(callee.object) && callee.object.name === emberImportAliasName) {
+          return;
+        }
 
         if (isIdentifier(method) && avoidedMethods.indexOf(method.name) > -1) {
           report(node);


Test patch:
diff --git a/tests/lib/rules/use-ember-get-and-set.js b/tests/lib/rules/use-ember-get-and-set.js
index 177bff62..6120042d 100644
--- a/tests/lib/rules/use-ember-get-and-set.js
+++ b/tests/lib/rules/use-ember-get-and-set.js
@@ -14,8 +14,8 @@ eslintTester.run('use-ember-get-and-set', rule, {
   valid: [
     'get(this, "test")',
     'get(controller, "test")',
-    'set(this, "test")',
-    'set(controller, "test")',
+    'set(this, "test", someValue)',
+    'set(controller, "test", someValue)',
     'getProperties(this, name, email, password)',
     'getProperties(controller, name, email, password)',
     'setProperties(this, {name: "Jon", email: "jon@snow.com"})',
@@ -34,6 +34,18 @@ eslintTester.run('use-ember-get-and-set', rule, {
       code: 'this.get("/resources")',
       filename: 'app/mirage/config.js',
     },
+    {
+      code: 'import Ember from "ember"; Ember.get(this, "test")',
+      parserOptions: { ecmaVersion: 6, sourceType: 'module' },
+    },
+    {
+      code: 'import Ember from "ember"; Ember.set(this, "test", someValue)',
+      parserOptions: { ecmaVersion: 6, sourceType: 'module' },
+    },
+    {
+      code: 'import EmberAlias from "ember"; EmberAlias.get(this, "test")',
+      parserOptions: { ecmaVersion: 6, sourceType: 'module' },
+    },
   ],
   invalid: [
     // Non-fixable errors
Repository: go-playground/validator (go)
Issue: Erroneous url_encoded validation
### Package version eg. v9, v10: 
v10

### Issue, Question or Enhancement:
Issue


### Code sample, to showcase or reproduce:

```go
	createOrganization := CreateOrganization{Organization: "hello"}
	err := validator.New().Struct(&createOrganization)
	if err != nil {
		var validationErrors validator.ValidationErrors
		var ok bool
		if validationErrors, ok = err.(validator.ValidationErrors); !ok {
			fmt.Println("non validation er")
			return
		}

		fmt.Printf("%v", validationErrors)
	} else {
		fmt.Printf("No validation error")
	}
```
Playground URL: https://play.golang.org/p/-awW91lwR-v 

If I use just english text, the above program should pass but it seem to fail. If I use an online url encoder program to encode an english text, I get the same input english text as the output. So the string `hello` should be considered as a valid url_encoded text.

However, if I use something else like `Hello+G%C3%BCnter` for the `Organization`, then the program works fine.
Pull request description: fix url_encoded regex
## Fixes Or Enhances
Attempt to fix url_encoded regex fixes #723 

**Make sure that you've checked the boxes below before you submit PR:**
- [x] Tests exist or have been written that cover this particular change.

@go-playground/validator-maintainers

Patch:
diff --git a/regexes.go b/regexes.go
index 9ecf4b1..ddcf785 100644
--- a/regexes.go
+++ b/regexes.go
@@ -45,7 +45,7 @@ const (
 	ethAddressRegexString            = `^0x[0-9a-fA-F]{40}$`
 	ethAddressUpperRegexString       = `^0x[0-9A-F]{40}$`
 	ethAddressLowerRegexString       = `^0x[0-9a-f]{40}$`
-	uRLEncodedRegexString            = `(%[A-Fa-f0-9]{2})`
+	uRLEncodedRegexString            = `^(?:[^%]|%[0-9A-Fa-f]{2})*$`
 	hTMLEncodedRegexString           = `&#[x]?([0-9a-fA-F]{2})|(&gt)|(&lt)|(&quot)|(&amp)+[;]?`
 	hTMLRegexString                  = `<[/]?([a-zA-Z]+).*?>`
 	splitParamsRegexString           = `'[^']*'|\S+`


Test patch:
diff --git a/validator_test.go b/validator_test.go
index db5bf73..5dcd00e 100644
--- a/validator_test.go
+++ b/validator_test.go
@@ -9537,6 +9537,9 @@ func TestURLEncodedValidation(t *testing.T) {
 		{"a%b", false},
 		{"1%2", false},
 		{"%%a%%", false},
+		{"hello", true},
+		{"", true},
+		{"+", true},
 	}
 
 	validate := New()
Repository: alecthomas/kong (go)
Issue: panic on Run with no selected nodes
If there's an application with no nodes selected, instead of returning an error, `Context.Run` panics.
Reproduction:

```go
func TestRegression(t *testing.T) {
        var emptyCLI struct{}
        parser, err := kong.New(&emptyCLI)
        assert.NoError(t, err)

        kctx, err := parser.Parse([]string{})
        assert.NoError(t, err)

        err = kctx.Run()
        assert.Error(t, err)
        assert.Contains(t, err.Error(), "no command selected")
}
```

This test passes against v1.2.1 but not against 1.6.0.
`git bisect` indicates that the issue was introduced in 2544d3f00855f6857cf121ce04608296b5858e4a.
Pull request description: fix(Context.Run): Don't panic on unselected root node
Fixes bug introduced in 2544d3f00855f6857cf121ce04608296b5858e4a.

That commit refactored control flow slightly and missed a case
where the selected node is an application node but it does not
have a Run method.

The fix was straightforward:
Treat the "if application node" code path as a possible way to fill
`node`, and check it again afterwards.

Resolves #483


Patch:
diff --git a/context.go b/context.go
index 4fd2302..b6a56e3 100644
--- a/context.go
+++ b/context.go
@@ -827,7 +827,9 @@ func (c *Context) Run(binds ...any) (err error) {
 			if method.IsValid() {
 				node = selected
 			}
-		} else {
+		}
+
+		if node == nil {
 			return fmt.Errorf("no command selected")
 		}
 	}


Test patch:
diff --git a/kong_test.go b/kong_test.go
index 2f3ab47..a7c03b2 100644
--- a/kong_test.go
+++ b/kong_test.go
@@ -2495,3 +2495,16 @@ func TestPrefixXorIssue343(t *testing.T) {
 	_, err = kctx.Parse([]string{"--source-password-file=foo", "--source-password=bar"})
 	assert.Error(t, err)
 }
+
+func TestIssue483EmptyRootNodeNoRun(t *testing.T) {
+	var emptyCLI struct{}
+	parser, err := kong.New(&emptyCLI)
+	assert.NoError(t, err)
+
+	kctx, err := parser.Parse([]string{})
+	assert.NoError(t, err)
+
+	err = kctx.Run()
+	assert.Error(t, err)
+	assert.Contains(t, err.Error(), "no command selected")
+}