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

nielstron / opshin
91%
dev: 92%

Build:
Build:
LAST BUILD BRANCH: openhands-fix-issue-5-try2
DEFAULT BRANCH: dev
Repo Added 31 May 2025 08:45AM UTC
Token jNDJRc3bZX8Vi3prW6HCOKB0a11eYpQt2 regen
Build 13 Last
Files 50
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

LAST BUILD ON BRANCH openhands-fix-issue-3-try1
branch: openhands-fix-issue-3-try1
CHANGE BRANCH
x
Reset
Sync Branches
  • openhands-fix-issue-3-try1
  • dev
  • fix/filter_type_check
  • main
  • openhands-fix-issue-1-try2
  • openhands-fix-issue-5-try2

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

Relevant lines Covered
Build:
Build:
5033 RELEVANT LINES 4765 COVERED LINES
1.87 HITS PER LINE
Source Files on openhands-fix-issue-3-try1
  • Tree
  • List 50
  • Changed 0
  • Source Changed 0
  • Coverage Changed 0
Coverage ∆ File Lines Relevant Covered Missed Hits/Line Branch Hits Branch Misses

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
17456116395 openhands-fix-issue-3-try1 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 li... push 04 Sep 2025 07:23AM UTC openhands-agent github
90.9
See All Builds (11)

Badge your Repo: opshin

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.

Could not find badge in README.

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

Refresh
  • Settings
  • Repo on GitHub
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