Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

LocalSkillSource.listResources returns backslash-separated paths on Windows

オープン
#1,541 コメント 1 件 リアクション 0 件 担当者 1 名 GitHub で見る

@hemasekhar-p がすでに取り組んでいます。

2026年9月24日 から。

評価

この issue はまだ評価されていません。

説明

needs review

🔴 Required Information

Describe the Bug:

LocalSkillSource.listResources(skillName, resourceDirectory) turns each resource path, relative to the skill directory, into a string with Path.toString(), which uses the platform separator. On Windows it returns assets\file1.txt, while ClassPathSkillSource and InMemorySkillSource return assets/file1.txt for the same skill layout. So the result of SkillSource.listResources depends on both the implementation and the OS, and LocalSkillSourceTest.testListResources fails on Windows.

Steps to Reproduce:

  1. On Windows, check out main (4092a1f).
  2. Run ./mvnw -pl core test -Dtest=LocalSkillSourceTest.
  3. testListResources fails; see the log below.

Expected Behavior:

[assets/file1.txt, assets/subdir/file2.txt]: /-separated paths, as ClassPathSkillSource and InMemorySkillSource return and as the test expects. adk-python keys directory-loaded skill resources the same way since google/adk-python@bc2c97c ("Key directory-loaded skill resources with forward slashes").

Observed Behavior:

[assets\file1.txt, assets\subdir\file2.txt]

Environment Details:

  • ADK Library Version (see maven dependency): main at 4092a1f (1.10.1). The code has not changed since LocalSkillSource was added in 1.3.0.
  • OS: Windows 11
  • TS Version (tsc --version): N/A (Java: Microsoft OpenJDK 17.0.19; Maven 4.0.0-rc-3 via mvnw)

Model Information:

  • Which model is being used: N/A

🟡 Optional Information

Regression:

No. LocalSkillSource has used Path.toString() here since it was added in 1.3.0.

Logs:

[ERROR] Failures: 
[ERROR]   LocalSkillSourceTest.testListResources:96 value of      : blockingGet()
missing (2)   : assets/file1.txt, assets/subdir/file2.txt
unexpected (2): assets\file1.txt, assets\subdir\file2.txt
---
expected      : [assets/file1.txt, assets/subdir/file2.txt]
but was       : [assets\file1.txt, assets\subdir\file2.txt]
[ERROR] Tests run: 21, Failures: 1, Errors: 0, Skipped: 0

Additional Context:

CI runs only on Ubuntu, where Path.toString() already uses /, so the test passes there. I have a one-line fix ready and will link the PR here.

How often has this issue occurred?:

  • Always (100%) on Windows
主要言語
Java
スター
1.7k
フォーク
421
平均マージ
3日 10時間
マージ済み PR(30日)
34

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

google/adk-java のほかの issue

google/adk-java の issue をすべて見る

似ている issue

Java の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。