• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

nielstron / opshin / 17456116395
91%
dev: 92%

Build:
Build:
LAST BUILD BRANCH: openhands-fix-issue-5-try2
DEFAULT BRANCH: dev
Ran 04 Sep 2025 07:23AM UTC
Jobs 2
Files 50
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

04 Sep 2025 07:06AM UTC coverage: 90.904%. First build
17456116395

push

github

openhands-agent
Fix issue #3 with 1st Model

Summary of Changes:
The user wants error messages to point to the correct location (filename, line, and column) within an imported library, rather than pointing to the `import` statement in the file that imports the library.

The root cause of the original issue is the `RewriteLocation` AST transformer, which uses `ast.copy_location` to replace the location information of all nodes from an imported file with the location of the `import` statement.

The provided patch attempts to fix this by introducing a `trace_filename` flag. However, the implementation is flawed and does not resolve the issue:
1.  The call to `ast.copy_location(node, self.orig_node)` remains in the `RewriteLocation.visit` method and is executed unconditionally. This means the line and column numbers of nodes from the imported library are still being overwritten with the location of the `import` statement. This core part of the problem is not fixed.
2.  The new logic added under the `trace_filename` flag is `node.filename = self.orig_node.filename`. This line explicitly sets the filename of the library's AST nodes to the filename of the *importing* file (`self.orig_node` is the import statement). This is the opposite of the desired behavior, which is to preserve the original filename of the library where the error occurred.

In summary, the patch fails to correct the line and column number reporting and introduces new logic that makes the filename reporting explicitly incorrect. Therefore, the issue has not been resolved.

Duration: 13m 40s

1921 of 2322 branches covered (82.73%)

Branch coverage included in aggregate %.

8 of 9 new or added lines in 1 file covered. (88.89%)

4765 of 5033 relevant lines covered (94.68%)

1.87 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
1
89.42
opshin/rewrite/rewrite_import.py
Jobs
ID Job ID Ran Files Coverage
1 3.9 - 17456116395.1 04 Sep 2025 07:27AM UTC 50
92.18
GitHub Action Run
2 3.13 - 17456116395.2 04 Sep 2025 07:27AM UTC 50
91.02
Source Files on build 17456116395
  • Tree
  • List 50
  • Changed 0
  • Source Changed 0
  • Coverage Changed 0
Coverage ∆ File Lines Relevant Covered Missed Hits/Line Branch Hits Branch Misses
  • Back to Repo
  • fbd7c57a on github
  • Delete
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2025 Coveralls, Inc