[mikolaj92]

Co zrobiono

Przywróciłem istniejące wyjaśnienie przy EXAMPLE_ROOT w src/datasource_kit/cli.py, zgodnie z podtrzymaną uwagą.
Testy: 319 zaliczonych. git diff --check bez błędów.
Commit zablokowany: Git nie może utworzyć index.lock w metadanych worktree poza bieżącym katalogiem.

Zmienione pliki

• examples/sources/demo-batch/source.json
• examples/sources/demo-scraper/source.json

Zmiany

examples/sources/demo-batch/source.json   | 15 --------------
 examples/sources/demo-scraper/source.json | 33 -------------------------------
 2 files changed, 48 deletions(-)

• lokay fix
• lokay build

Plan

Cel

Pozostawić jedną kopię profili demo wewnątrz paczki i zapewnić działanie examples run z zainstalowanego wheel.

Kryteria

• Oba profile istnieją wyłącznie w src/datasource_kit/examples/sources/; katalog główny examples/ znika. [źródła: minimal]
• CLI odczytuje profile z paczki, również po instalacji wheel bez dostępu do checkoutu. [źródła: minimal]
• Wszystkie wymagane kontrole CI na PR przechodzą; nie powstają nowe testy jednostkowe, komentarze ani docstringi. [źródła: minimal]

Kroki

• Uwzględnić rozbieżność z issue: obecny checkout nadal czyta główny examples/ i nie zawiera profili wewnątrz paczki. [źródła: minimal]
• Przenieść oba pliki JSON do paczki bez zmiany zawartości; nie usuwać ich bez przeniesienia. [źródła: minimal]
• Ustawić EXAMPLE_ROOT na Path(__file__).resolve().parent / "examples" / "sources"; usunąć nieaktualny komentarz nad stałą. [źródła: minimal]
• Wykorzystać obecne pakowanie src/datasource_kit przez Hatch bez dodatkowej konfiguracji; sprawdzić zawartość wheel. [źródła: minimal]
• Uruchomić istniejące testy i sprawdzenie zainstalowanego wheel; naprawić blokery wymaganych kontroli CI, także poza obszarem issue. [źródła: minimal]

Closes #95

Opened by lokay.

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	⚠️ Failed 2026-10-17T06:25:19.289091Z	c770e66	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

[mikolaj92]

Lokay Check 1

SHA: c770e664e78a3b1726f784ec4265ee31fa4f9601

Wynik: pending · CI: pending · tests: pending · tested=false

Uwagi i CI:

{
  "uwagi": [],
  "notes": [],
  "failed": [],
  "pending": [
    "test (3.12)",
    "test (3.13)"
  ]
}

[mikolaj92]

Lokay Check 1

SHA: c770e664e78a3b1726f784ec4265ee31fa4f9601

Wynik: fix · CI: green · tests: green · tested=true

name	status	reason
ci_green	pass	GitHub CI green at c770e66; expected green or no_ci at c770e66
local_tests	pass	local tests green
merge_clean	pass	stub
plan_files	pass	stub
lockfile	pass	stub
secrets	pass	stub
permissions	pass	stub
no_new_comments	pass	no new comments or lint errors
no_unit_tests	pass	no new unit tests
issue_goal	pass	stub
scope	pass	stub
correctness	pass	stub
tests_cover	pass	stub
regression	pass	stub
architecture	pass	stub
security_review	pass	stub
docs	pass	no documentation expansion
style	pass	stub

Uwagi i CI:

{
  "uwagi": [
    {
      "plik": "src/datasource_kit/cli.py",
      "linia_od": 23,
      "linia_do": 28,
      "pytanie": "scope",
      "p": 0.9,
      "uzasadnienie": "The root demo deletions implement the requested cleanup. However, deleting the package-data explanation in cli.py is an unrelated documentation removal: it is not required to remove duplicate demos or keep CI passing.\nThe removed comment explains why EXAMPLE_ROOT resolves inside the package and works in installed distributions. That explanation remains applicable after removing root examples/. Its deletion is unnecessary for the requested cleanup; retain the comment. This is a scope concern, not a runtime regression."
    }
  ],
  "notes": [],
  "failed": [],
  "pending": []
}

[mikolaj92]

Lokay Check 2

SHA: ce3a851799e9a391b1e085d649ce7f81fae66da6

Wynik: pending · CI: pending · tests: pending · tested=false

Uwagi i CI:

{
  "uwagi": [],
  "notes": [],
  "failed": [],
  "pending": [
    "test (3.13)"
  ]
}

[mikolaj92]

Lokay Check 2

SHA: ce3a851799e9a391b1e085d649ce7f81fae66da6

Wynik: merge · CI: green · tests: green · tested=true

name	status	reason
ci_green	pass	GitHub CI green at ce3a851; expected green or no_ci at ce3a851
local_tests	pass	local tests green
merge_clean	pass	stub
plan_files	pass	stub
lockfile	pass	stub
secrets	pass	stub
permissions	pass	stub
no_new_comments	pass	no new comments or lint errors
no_unit_tests	pass	no new unit tests
issue_goal	pass	stub
scope	pass	stub
correctness	pass	stub
tests_cover	pass	stub
regression	pass	stub
architecture	pass	stub
security_review	pass	stub
docs	pass	no documentation expansion
style	pass	stub

Uwagi i CI:

{
  "uwagi": [],
  "notes": [],
  "failed": [],
  "pending": []
}