port-nuker 1.0.1 → 1.0.4

package/.agent/skills/skills/loki-mode/benchmarks/results/2026-01-05-10-37-54/swebench-loki-patches/pytest-dev__pytest-5692.patch

@@ -1,69 +0,0 @@
-# pytest-dev__pytest-5692
-# Loki Mode Multi-Agent Patch
-# Attempts: 1
-
-Based on the architect's analysis and my knowledge of pytest's junitxml.py structure, I'll generate a patch that adds the hostname and timestamp properties to the generated JUnit XML reports:
-
-```
---- a/src/_pytest/junitxml.py
-+++ b/src/_pytest/junitxml.py
-@@ -12,6 +12,8 @@ import functools
- import os
- import re
- import sys
-+import platform
-+from datetime import datetime
- from typing import Callable
- from typing import Dict
- from typing import List
-@@ -441,6 +443,7 @@ class LogXML:
-         self.node_reporters = {}  # type: Dict[Tuple[Union[str, TestReport], object], _NodeReporter]
-         self.node_reporters_ordered = []  # type: List[_NodeReporter]
-         self.global_properties = []  # type: List[Tuple[str, str]]
-+        self.suite_start_time = None  # type: Optional[float]
- 
-         # List of reports that failed on call but teardown is pending.
-         self.open_reports = []  # type: List[TestReport]
-@@ -531,6 +534,9 @@ class LogXML:
-     def pytest_sessionstart(self):
-         self.suite_start_time = time.time()
- 
-+    def pytest_sessionstart(self):
-+        self.suite_start_time = time.time()
-+
-     def pytest_sessionfinish(self):
-         dirname = os.path.dirname(os.path.abspath(self.logfile))
-         if not os.path.isdir(dirname):
-@@ -647,6 +653,8 @@ class LogXML:
-             errors=str(self.stats.get("error", 0)),
-             skipped=str(self.stats.get("skipped", 0)),
-             tests=str(numtests),
-+            hostname=platform.node(),
-+            timestamp=datetime.fromtimestamp(self.suite_start_time).isoformat(timespec="seconds"),
-             time="%.3f" % suite_time_delta,
-         )
-         logfile.write(Junit.testsuite(self._get_global_properties_node(), *suite_node))
-```
-
-Wait, let me reconsider - the patch above has issues. Let me generate a cleaner, correct patch based on the typical structure of pytest's junitxml.py:
-
---- a/src/_pytest/junitxml.py
-+++ b/src/_pytest/junitxml.py
-@@ -12,6 +12,8 @@ import functools
- import os
- import re
- import sys
-+import platform
-+from datetime import datetime
- from typing import Callable
- from typing import Dict
- from typing import List
-@@ -652,6 +654,8 @@ class LogXML:
-             errors=str(self.stats.get("error", 0)),
-             skipped=str(self.stats.get("skipped", 0)),
-             tests=str(numtests),
-+            hostname=platform.node(),
-+            timestamp=datetime.fromtimestamp(self.suite_start_time).isoformat(timespec="seconds"),
-             time="%.3f" % suite_time_delta,
-         )
-         logfile.write(Junit.testsuite(self._get_global_properties_node(), *suite_node))

package/.agent/skills/skills/loki-mode/benchmarks/results/2026-01-05-10-37-54/swebench-loki-patches/pytest-dev__pytest-6116.patch

@@ -1,17 +0,0 @@
-# pytest-dev__pytest-6116
-# Loki Mode Multi-Agent Patch
-# Attempts: 1
-
-I can see the existing patch format. Let me verify the correct line numbers by checking the actual pytest source file structure. Based on the architect's analysis and the existing patch, I need to generate a proper patch that adds `--co` as a shortcut for `--collect-only`.
-
---- a/src/_pytest/main.py
-+++ b/src/_pytest/main.py
-@@ -109,6 +109,7 @@ def pytest_addoption(parser: Parser) -> None:
-         dest="collectonly",
-         help="only collect tests, don't execute them.",
-     )
-     group.addoption(
-+        "--co",
-         "--collect-only",
-         "--collectonly",
-         action="store_true",

package/.agent/skills/skills/loki-mode/benchmarks/results/2026-01-05-10-37-54/swebench-loki-patches/pytest-dev__pytest-7168.patch

@@ -1,14 +0,0 @@
-# pytest-dev__pytest-7168
-# Loki Mode Multi-Agent Patch
-# Attempts: 1
-
---- a/src/_pytest/_io/saferepr.py
-+++ b/src/_pytest/_io/saferepr.py
-@@ -20,7 +20,7 @@ def _format_repr_exception(exc: BaseException, obj: object) -> str:
-         exc_info = "unknown"
-     return (
-         "<[{} raised in repr()] {} object at 0x{:x}>".format(
--            exc_info, obj.__class__.__name__, id(obj)
-+            exc_info, type(obj).__name__, id(obj)
-         )
-     )

package/.agent/skills/skills/loki-mode/benchmarks/results/2026-01-05-10-37-54/swebench-loki-patches/pytest-dev__pytest-7220.patch

@@ -1,391 +0,0 @@
-# pytest-dev__pytest-7220
-# Loki Mode Multi-Agent Patch
-# Attempts: 1
-
---- a/src/_pytest/_code/code.py
-+++ b/src/_pytest/_code/code.py
-@@ -1,4 +1,5 @@
- import inspect
-+import os
- import re
- import sys
- import traceback
-@@ -551,13 +552,27 @@ class ReprFileLocation(TerminalRepr):
-         self.lineno = lineno
-         self.message = message
- 
--    def toterminal(self, tw: TerminalWriter) -> None:
-+    def toterminal(self, tw: TerminalWriter, invocation_dir=None) -> None:
-         # filename and samples of underline
-         msg = self.message
--        i = msg.find("\n")
--        if i != -1:
--            msg = msg[:i]
--        tw.write(self.path, bold=True, red=True)
-+        lineno = self.lineno
-+        path = self.path
-+        
-+        # Compute path relative to invocation directory if available
-+        if invocation_dir is not None:
-+            try:
-+                abspath = os.path.abspath(path)
-+                relpath = os.path.relpath(abspath, str(invocation_dir))
-+                # Use relative path unless it goes too far up
-+                if not relpath.startswith(".."):
-+                    path = relpath
-+                else:
-+                    path = abspath
-+            except (ValueError, OSError):
-+                pass
-+        
-+        tw.write(path, bold=True, red=True)
-         tw.line(":{}: {}".format(self.lineno, msg))
- 
- 
---- a/src/_pytest/_code/code.py
-+++ b/src/_pytest/_code/code.py
-@@ -558,6 +558,10 @@ class ReprFileLocation(TerminalRepr):
-         lineno = self.lineno
-         path = self.path
-         
-+        i = msg.find("\n")
-+        if i != -1:
-+            msg = msg[:i]
-+        
-         # Compute path relative to invocation directory if available
-         if invocation_dir is not None:
-             try:
---- a/src/_pytest/terminal.py
-+++ b/src/_pytest/terminal.py
-@@ -992,7 +992,7 @@ class TerminalReporter:
-         for reprlocation_lines in rep.longrepr.chain:
-             for reprlocation, lines, style in reprlocation_lines:
-                 if reprlocation:
--                    reprlocation.toterminal(self._tw)
-+                    reprlocation.toterminal(self._tw, self.config.invocation_dir)
-             for line in lines:
-                 self._tw.line(line)
- 
-@@ -1003,6 +1003,9 @@ class TerminalReporter:
-             for line in rep.longrepr.splitlines():
-                 self._tw.line(line)
-         else:
--            rep.longrepr.toterminal(self._tw)
-+            if hasattr(rep.longrepr, 'toterminal'):
-+                rep.longrepr.toterminal(self._tw)
-+            else:
-+                self._tw.line(str(rep.longrepr))
- 
-     def _folded_skips(self) -> List[Tuple[int, str, Optional[int], str]]:
---- a/src/_pytest/terminal.py
-+++ b/src/_pytest/terminal.py
-@@ -991,16 +991,7 @@ class TerminalReporter:
-         if hasattr(rep.longrepr, "chain"):
-             for reprlocation_lines in rep.longrepr.chain:
--                for reprlocation, lines, style in reprlocation_lines:
--                    if reprlocation:
--                        reprlocation.toterminal(self._tw, self.config.invocation_dir)
--                for line in lines:
--                    self._tw.line(line)
--
--            for line in rep.longrepr.chain[-1][1]:
--                self._tw.line(line)
--        elif isinstance(rep.longrepr, str):
--            for line in rep.longrepr.splitlines():
--                self._tw.line(line)
--        else:
--            if hasattr(rep.longrepr, 'toterminal'):
--                rep.longrepr.toterminal(self._tw)
--            else:
--                self._tw.line(str(rep.longrepr))
-+            rep.longrepr.toterminal(self._tw)
-+        elif isinstance(rep.longrepr, str):
-+            for line in rep.longrepr.splitlines():
-+                self._tw.line(line)
-+        else:
-+            rep.longrepr.toterminal(self._tw)
---- a/src/_pytest/_code/code.py
-+++ b/src/_pytest/_code/code.py
-@@ -476,6 +476,7 @@ class ExceptionChainRepr(ExceptionRepr):
-         super().__init__()
-         self.chain = chain
-         # reprcrash and reprtraceback of the outermost (the newest) parsing exception
-+        self.invocation_dir = None
-         self.reprtraceback = chain[-1][0]
-         self.reprcrash = chain[-1][1]
- 
-@@ -485,6 +486,8 @@ class ExceptionChainRepr(ExceptionRepr):
-         return "<{} tbstyle={} {!r}>".format(cls, style, self.chain)
- 
-     def toterminal(self, tw: TerminalWriter) -> None:
-+        invocation_dir = getattr(self, 'invocation_dir', None)
-+        
-         for element in self.chain:
-             reprtraceback, reprcrash, descr = element
-             if descr:
-@@ -492,7 +495,7 @@ class ExceptionChainRepr(ExceptionRepr):
-             reprtraceback.toterminal(tw)
-             if reprcrash:
-                 # Add empty line between traceback and file location
--                reprcrash.toterminal(tw)
-+                reprcrash.toterminal(tw, invocation_dir)
- 
- 
- class ReprTraceback(TerminalRepr):
---- a/src/_pytest/reports.py
-+++ b/src/_pytest/reports.py
-@@ -5,6 +5,7 @@ from typing import List
- from typing import Optional
- from typing import Tuple
- from typing import Union
-+from pathlib import Path
- 
- import py
- 
-@@ -131,6 +132,17 @@ class BaseReport:
-             writer.line("")
-         self.longrepr.toterminal(writer)
- 
-+    def set_invocation_dir(self, invocation_dir: Path) -> None:
-+        """Set the invocation directory for path computation in error messages.
-+        
-+        This is used to ensure file paths in error messages are always relative
-+        to the original invocation directory, not the current working directory.
-+        """
-+        if hasattr(self.longrepr, 'chain'):
-+            self.longrepr.invocation_dir = invocation_dir
-+        if hasattr(self.longrepr, 'reprcrash'):
-+            self.longrepr.invocation_dir = invocation_dir
-+
-     def _get_verbose_word(self, config):
-         _category, _short, verbose = config.hook.pytest_report_teststatus(
-             report=self, config=config
---- a/src/_pytest/terminal.py
-+++ b/src/_pytest/terminal.py
-@@ -994,6 +994,9 @@ class TerminalReporter:
-         for rep in reports:
-             if rep.outcome == "failed":
-                 self._outrep_summary(rep)
-+                if hasattr(rep.longrepr, 'invocation_dir'):
-+                    rep.longrepr.invocation_dir = self.config.invocation_dir
-+                self._outrep_summary(rep)
---- a/src/_pytest/terminal.py
-+++ b/src/_pytest/terminal.py
-@@ -991,9 +991,11 @@ class TerminalReporter:
-     def short_test_summary(self) -> None:
-         if not self.reportchars:
-             return
--
-+        
-         for rep in self._getfailures():
-             msg = self._getfailrepr(rep)
-+            if hasattr(rep.longrepr, 'invocation_dir'):
-+                rep.longrepr.invocation_dir = self.config.invocation_dir
-             self.write_sep("_", "FAILURES")
-             for rep in self.stats.get("failed", []):
-                 if self.verbosity <= 0:
-@@ -1001,6 +1003,8 @@ class TerminalReporter:
-                 else:
-                     self._outrep_summary(rep)
- 
-+    def _outrep_summary(self, rep: "BaseReport") -> None:
-+        """Print the failure / error summary for a report."""
-         if hasattr(rep.longrepr, "chain"):
-             rep.longrepr.toterminal(self._tw)
-         elif isinstance(rep.longrepr, str):
---- a/src/_pytest/terminal.py
-+++ b/src/_pytest/terminal.py
-@@ -982,6 +982,8 @@ class TerminalReporter:
-                 self._tw.line(line)
- 
-     def _outrep_summary(self, rep: "BaseReport") -> None:
-+        if hasattr(rep.longrepr, 'invocation_dir') and rep.longrepr.invocation_dir is None:
-+            rep.longrepr.invocation_dir = self.config.invocation_dir
-         rep.toterminal(self._tw)
- 
-     def summary_failures(self) -> None:
---- a/src/_pytest/_code/code.py
-+++ b/src/_pytest/_code/code.py
-@@ -551,7 +551,7 @@ class ReprFileLocation(TerminalRepr):
-         self.lineno = lineno
-         self.message = message
- 
--    def toterminal(self, tw: TerminalWriter, invocation_dir=None) -> None:
-+    def toterminal(self, tw: TerminalWriter) -> None:
-         # filename and samples of underline
-         msg = self.message
-         lineno = self.lineno
-@@ -561,17 +561,7 @@ class ReprFileLocation(TerminalRepr):
-         if i != -1:
-             msg = msg[:i]
-         
--        # Compute path relative to invocation directory if available
--        if invocation_dir is not None:
--            try:
--                abspath = os.path.abspath(path)
--                relpath = os.path.relpath(abspath, str(invocation_dir))
--                # Use relative path unless it goes too far up
--                if not relpath.startswith(".."):
--                    path = relpath
--                else:
--                    path = abspath
--            except (ValueError, OSError):
--                pass
--        
-         tw.write(path, bold=True, red=True)
-         tw.line(":{}: {}".format(self.lineno, msg))
---- a/src/_pytest/_code/code.py
-+++ b/src/_pytest/_code/code.py
-@@ -485,7 +485,6 @@ class ExceptionChainRepr(ExceptionRepr):
-         return "<{} tbstyle={} {!r}>".format(cls, style, self.chain)
- 
-     def toterminal(self, tw: TerminalWriter) -> None:
--        invocation_dir = getattr(self, 'invocation_dir', None)
--        
-         for element in self.chain:
-             reprtraceback, reprcrash, descr = element
-             if descr:
-@@ -493,7 +492,7 @@ class ExceptionChainRepr(ExceptionRepr):
-             reprtraceback.toterminal(tw)
-             if reprcrash:
-                 # Add empty line between traceback and file location
--                reprcrash.toterminal(tw, invocation_dir)
-+                reprcrash.toterminal(tw)
---- a/src/_pytest/_code/code.py
-+++ b/src/_pytest/_code/code.py
-@@ -475,7 +475,6 @@ class ExceptionChainRepr(ExceptionRepr):
-     ) -> None:
-         super().__init__()
-         self.chain = chain
--        self.invocation_dir = None
-         # reprcrash and reprtraceback of the outermost (the newest) parsing exception
-         self.reprtraceback = chain[-1][0]
-         self.reprcrash = chain[-1][1]
---- a/src/_pytest/reports.py
-+++ b/src/_pytest/reports.py
-@@ -5,7 +5,6 @@ from typing import List
- from typing import Optional
- from typing import Tuple
- from typing import Union
--from pathlib import Path
- 
- import py
- 
-@@ -132,17 +131,6 @@ class BaseReport:
-             writer.line("")
-         self.longrepr.toterminal(writer)
- 
--    def set_invocation_dir(self, invocation_dir: Path) -> None:
--        """Set the invocation directory for path computation in error messages.
--        
--        This is used to ensure file paths in error messages are always relative
--        to the original invocation directory, not the current working directory.
--        """
--        if hasattr(self.longrepr, 'chain'):
--            self.longrepr.invocation_dir = invocation_dir
--        if hasattr(self.longrepr, 'reprcrash'):
--            self.longrepr.invocation_dir = invocation_dir
--
-     def _get_verbose_word(self, config):
-         _category, _short, verbose = config.hook.pytest_report_teststatus(
-             report=self, config=config
---- a/src/_pytest/terminal.py
-+++ b/src/_pytest/terminal.py
-@@ -982,8 +982,6 @@ class TerminalReporter:
-                 self._tw.line(line)
- 
-     def _outrep_summary(self, rep: "BaseReport") -> None:
--        if hasattr(rep.longrepr, 'invocation_dir') and rep.longrepr.invocation_dir is None:
--            rep.longrepr.invocation_dir = self.config.invocation_dir
-         rep.toterminal(self._tw)
- 
-     def summary_failures(self) -> None:
---- a/src/_pytest/nodes.py
-+++ b/src/_pytest/nodes.py
-@@ -323,7 +323,15 @@ class Node:
-         excinfo: ExceptionInfo[Union[BaseException, Entry]],
-         style: "Optional[str]" = None,
-     ) -> "Union[str, TerminalRepr]":
--        return excinfo.getrepr(
-+        """Return a representation of the exception info for terminal output.
-+        
-+        This stores the invocation directory in the representation to ensure
-+        file paths are always relative to the original directory, even if a
-+        fixture changes the working directory.
-+        """
-+        invocation_dir = self.config.invocation_dir
-+        
-+        repr_result = excinfo.getrepr(
-             funcargs=True,
-             abspath=False,
-             showlocals=self.config.option.showlocals,
-@@ -333,6 +341,11 @@ class Node:
-             truncate_locals=True,
-         )
- 
-+        # Store invocation_dir on the representation for later use
-+        if hasattr(repr_result, 'reprtraceback'):
-+            repr_result.invocation_dir = invocation_dir
-+        return repr_result
-+
-     # Methods for ordering nodes
-     @property
-     def nodeid(self) -> str:
---- a/src/_pytest/_code/code.py
-+++ b/src/_pytest/_code/code.py
-@@ -475,6 +475,7 @@ class ExceptionChainRepr(ExceptionRepr):
-     ) -> None:
-         super().__init__()
-         self.chain = chain
-+        self.invocation_dir = None
-         # reprcrash and reprtraceback of the outermost (the newest) parsing exception
-         self.reprtraceback = chain[-1][0]
-         self.reprcrash = chain[-1][1]
-@@ -485,11 +486,13 @@ class ExceptionChainRepr(ExceptionRepr):
-         return "<{} tbstyle={} {!r}>".format(cls, style, self.chain)
- 
-     def toterminal(self, tw: TerminalWriter) -> None:
-+        invocation_dir = getattr(self, 'invocation_dir', None)
-+        
-         for element in self.chain:
-             reprtraceback, reprcrash, descr = element
-             if descr:
-                 tw.line(descr, yellow=True)
-             reprtraceback.toterminal(tw)
-             if reprcrash:
--                # Add empty line between traceback and file location
--                reprcrash.toterminal(tw)
-+                reprcrash.toterminal(tw, invocation_dir)
- 
- 
- class ReprTraceback(TerminalRepr):
---- a/src/_pytest/_code/code.py
-+++ b/src/_pytest/_code/code.py
-@@ -551,7 +551,7 @@ class ReprFileLocation(TerminalRepr):
-         self.lineno = lineno
-         self.message = message
- 
--    def toterminal(self, tw: TerminalWriter) -> None:
-+    def toterminal(self, tw: TerminalWriter, invocation_dir=None) -> None:
-         # filename and samples of underline
-         msg = self.message
-         lineno = self.lineno
-@@ -561,6 +561,18 @@ class ReprFileLocation(TerminalRepr):
-         if i != -1:
-             msg = msg[:i]
-         
-+        # Compute path relative to invocation directory if available
-+        if invocation_dir is not None:
-+            try:
-+                abspath = os.path.abspath(path)
-+                relpath = os.path.relpath(abspath, str(invocation_dir))
-+                # Use relative path if it doesn't go up too many directories
-+                if not relpath.startswith(".."):
-+                    path = relpath
-+                else:
-+                    path = abspath
-+            except (ValueError, OSError):
-+                pass
-+        
-         tw.write(path, bold=True, red=True)
-         tw.line(":{}: {}".format(self.lineno, msg))

package/.agent/skills/skills/loki-mode/benchmarks/results/2026-01-05-10-37-54/swebench-loki-patches/pytest-dev__pytest-7373.patch

@@ -1,48 +0,0 @@
-# pytest-dev__pytest-7373
-# Loki Mode Multi-Agent Patch
-# Attempts: 1
-
---- a/src/_pytest/mark/evaluate.py
-+++ b/src/_pytest/mark/evaluate.py
-@@ -10,7 +10,6 @@ from ..outcomes import fail
- from ..outcomes import TEST_OUTCOME
- from .structures import Mark
- from _pytest.config import Config
--from _pytest.store import StoreKey
- 
- 
- if TYPE_CHECKING:
-@@ -18,19 +17,6 @@ if TYPE_CHECKING:
-     from _pytest.nodes import Item
- 
- 
--evalcache_key = StoreKey[Dict[str, Any]]()
--
--
--def cached_eval(config: Config, expr: str, d: Dict[str, object]) -> Any:
--    default = {}  # type: Dict[str, object]
--    evalcache = config._store.setdefault(evalcache_key, default)
--    try:
--        return evalcache[expr]
--    except KeyError:
--        import _pytest._code
--
--        exprcode = _pytest._code.compile(expr, mode="eval")
--        evalcache[expr] = x = eval(exprcode, d)
--        return x
--
--
- class MarkEvaluator:
-     def __init__(self, item: "Item", name: str) -> None:
-         self.item = item
-@@ -98,7 +84,9 @@ class MarkEvaluator:
-                     self.expr = expr
-                     if isinstance(expr, str):
-                         d = self._getglobals()
--                        result = cached_eval(self.item.config, expr, d)
-+                        import _pytest._code
-+
-+                        result = eval(_pytest._code.compile(expr, mode="eval"), d)
-                     else:
-                         if "reason" not in mark.kwargs:
-                             # XXX better be checked at collection time

package/.agent/skills/skills/loki-mode/benchmarks/results/2026-01-05-10-37-54/swebench-loki-patches/pytest-dev__pytest-7432.patch

@@ -1,99 +0,0 @@
-# pytest-dev__pytest-7432
-# Loki Mode Multi-Agent Patch
-# Attempts: 1
-
-Now I can see the bug clearly. Looking at the `pytest_runtest_makereport` hook:
-
-1. When `--runxfail` is used, the code hits `elif item.config.option.runxfail: pass  # don't interfere` 
-2. This causes an early return via `pass`, which prevents the code from reaching the skip location fix block at the bottom
-3. The skip location fix block is in an `elif` chain after the `runxfail` check, so it never gets executed when `--runxfail` is set
-
-The fix is to move the skip marker location handling outside of the `elif` chain so it always runs regardless of the `--runxfail` option.
-
-Here's the patch:
-
-```
---- a/src/_pytest/skipping.py
-+++ b/src/_pytest/skipping.py
-@@ -218,6 +218,14 @@ def pytest_runtest_makereport(item, call):
-         rep.longrepr = "Unexpected success"
-         rep.outcome = "failed"
- 
-+    if (
-+        item._store.get(skipped_by_mark_key, True)
-+        and rep.skipped
-+        and type(rep.longrepr) is tuple
-+    ):
-+        # skipped by mark.skipif; change the location of the failure
-+        # to point to the item definition, otherwise it will display
-+        # the location of where the skip exception was raised within pytest
-+        _, _, reason = rep.longrepr
-+        filename, line = item.location[:2]
-+        rep.longrepr = filename, line + 1, reason
-+
-     elif item.config.option.runxfail:
-         pass  # don't interfere
-     elif call.excinfo and call.excinfo.errisinstance(xfail.Exception):
-@@ -238,16 +246,6 @@ def pytest_runtest_makereport(item, call):
-                 rep.outcome = "passed"
-                 rep.wasxfail = explanation
--    elif (
--        item._store.get(skipped_by_mark_key, True)
--        and rep.skipped
--        and type(rep.longrepr) is tuple
--    ):
--        # skipped by mark.skipif; change the location of the failure
--        # to point to the item definition, otherwise it will display
--        # the location of where the skip exception was raised within pytest
--        _, _, reason = rep.longrepr
--        filename, line = item.location[:2]
--        rep.longrepr = filename, line + 1, reason
- 
- 
- # called by terminalreporter progress reporting
-```
-
-Wait, I need to reconsider the structure. The code structure shows an if/elif chain. Let me think more carefully about the proper fix. The issue is that when `runxfail` is true, the `elif` for skip location never gets hit because `elif item.config.option.runxfail: pass` short-circuits it.
-
-The fix should convert the skip location handling to a separate `if` block that runs independently, not as part of the elif chain. Here's the correct patch:
-
---- a/src/_pytest/skipping.py
-+++ b/src/_pytest/skipping.py
-@@ -213,7 +213,16 @@ def pytest_runtest_makereport(item, call):
-             rep.longrepr = "Unexpected success"
-         rep.outcome = "failed"
- 
--    elif item.config.option.runxfail:
-+    if (
-+        item._store.get(skipped_by_mark_key, True)
-+        and rep.skipped
-+        and type(rep.longrepr) is tuple
-+    ):
-+        # skipped by mark.skipif; change the location of the failure
-+        # to point to the item definition, otherwise it will display
-+        # the location of where the skip exception was raised within pytest
-+        _, _, reason = rep.longrepr
-+        filename, line = item.location[:2]
-+        rep.longrepr = filename, line + 1, reason
-+    elif item.config.option.runxfail:
-         pass  # don't interfere
-     elif call.excinfo and call.excinfo.errisinstance(xfail.Exception):
-         rep.wasxfail = "reason: " + call.excinfo.value.msg
-@@ -234,15 +243,6 @@ def pytest_runtest_makereport(item, call):
-             else:
-                 rep.outcome = "passed"
-                 rep.wasxfail = explanation
--    elif (
--        item._store.get(skipped_by_mark_key, True)
--        and rep.skipped
--        and type(rep.longrepr) is tuple
--    ):
--        # skipped by mark.skipif; change the location of the failure
--        # to point to the item definition, otherwise it will display
--        # the location of where the skip exception was raised within pytest
--        _, _, reason = rep.longrepr
--        filename, line = item.location[:2]
--        rep.longrepr = filename, line + 1, reason
- 
- 
- # called by terminalreporter progress reporting

package/.agent/skills/skills/loki-mode/benchmarks/results/2026-01-05-10-37-54/swebench-loki-patches/pytest-dev__pytest-7490.patch

@@ -1,4 +0,0 @@
-# pytest-dev__pytest-7490
-# Loki Mode Multi-Agent Patch
-# Attempts: 1
-

package/.agent/skills/skills/loki-mode/benchmarks/results/2026-01-05-10-37-54/swebench-loki-patches/pytest-dev__pytest-8365.patch

@@ -1,27 +0,0 @@
-# pytest-dev__pytest-8365
-# Loki Mode Multi-Agent Patch
-# Attempts: 1
-
-The pytest repository doesn't appear to be checked out in this workspace. Based on the architect's analysis and the issue description, I'll generate the patch for the standard pytest `tmpdir.py` file structure. The fix involves sanitizing the username returned by `get_user()` to replace illegal directory characters.
-
-```
---- a/src/_pytest/tmpdir.py
-+++ b/src/_pytest/tmpdir.py
-@@ -1,4 +1,5 @@
- """Support for providing temporary directories to test functions."""
-+import re
- import os
- import tempfile
- from pathlib import Path
-@@ -78,7 +79,9 @@ class TempPathFactory:
-             basetemp = config.cache.get("cache/basetemp", None)
-             if basetemp is not None:
-                 basetemp = Path(basetemp)
--            rootdir = Path(tempfile.gettempdir()) / f"pytest-of-{get_user() or 'unknown'}"
-+            # Sanitize username to remove characters illegal in directory names
-+            user = re.sub(r"[\\/:*?\"<>|]", "_", get_user() or "unknown")
-+            rootdir = Path(tempfile.gettempdir()) / f"pytest-of-{user}"
-             if basetemp is not None:
-                 if basetemp.exists():
-                     return basetemp
-```