docs(policy): record that only a dataset declaring its terms can be published - #801
Merged
Merged
Conversation
yeongseon
requested changes
Oct 4, 2026
yeongseon
left a comment
Collaborator
There was a problem hiding this comment.
리뷰 — 수정 요청 한 가지.
결정에는 동의합니다: 조건을 선언한 데이터셋만 공개 게시합니다. 확인하지 않은 KOGL 유형을 기관 기본값으로 채우지 않는다는 근거도 맞습니다.
고칠 것: tests/unit/test_declared_terms_decision.py 에 # type: ignore[type-arg] 가 4곳 있습니다 (refs() -> list, 그리고 refs: list 인자 3곳). AGENTS.md 가 type: ignore 를 금지합니다 — list[DatasetRef] 로 적으면 ignore 없이 됩니다. CI 의 mypy 는 src 만 보기 때문에 걸리지 않았을 뿐입니다.
main 과 충돌 상태(CHANGELOG)이니 같이 리베이스해 주세요.
Eomdahyeon
force-pushed
the
docs/issue-785-publish-needs-declared-terms
branch
from
October 4, 2026 14:43
6469c44 to
18e3e42
Compare
Collaborator
Author
|
리뷰 반영: |
yeongseon
approved these changes
Oct 4, 2026
yeongseon
left a comment
Collaborator
There was a problem hiding this comment.
수정 확인했습니다 — type: ignore 4곳이 list[DatasetRef] 로 바뀌었고 그 밖의 내용은 그대로입니다. 체크도 통과했습니다. 승인합니다.
Eomdahyeon
added a commit
that referenced
this pull request
Oct 4, 2026
## 문제 #801 리뷰가 짚은 것과 같은 위반이 오늘 내가 이미 머지한 테스트에 세 곳 더 있다. AGENTS.md 는 `type: ignore` 를 금지한다 ("Fix the type error instead of silencing it"). - `tests/unit/test_nodata_is_an_empty_result.py` (#792): `representation="api_json" # type: ignore[arg-type]` - `tests/unit/core/test_list_all_memory.py` (#802): `_make_executor(transport) # type: ignore[arg-type]` 두 곳 ## 변경 내용 - 문자열 대신 `Representation.API_JSON` 을 넘긴다. - `_LazyTransport` 를 `FakeTransport` 의 서브클래스로 만들어 `_make_executor` 가 받는 타입이 되게 했다. 테스트 동작은 그대로다: 두 파일 `13 passed`. ruff 통과. ## 범위 오늘 내가 추가한 파일만 고쳤다. `main` 의 `tests/` 에는 `type: ignore` 가 그 밖에도 53곳 남아 있다(`git grep -c` 합계 56 중 이 3곳 제외) — 그것들은 건드리지 않았다. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Eomdahyeon <213566566+Eomdahyeon@users.noreply.github.com> Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Eomdahyeon
force-pushed
the
docs/issue-785-publish-needs-declared-terms
branch
from
October 4, 2026 15:18
18e3e42 to
95fff4e
Compare
…ublished Closes #785 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Eomdahyeon
force-pushed
the
docs/issue-785-publish-needs-declared-terms
branch
from
October 4, 2026 15:28
95fff4e to
adacbe2
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #785
결정 — (b)
이슈는 (a) 나머지 131개의 라이선스를 채우거나 (b) "spec 기반 데이터셋만 publish 할 수 있다"를 계약으로 적으라고 했다. (b)를 택했다.
license가 없는 데이터셋은DatasetRef.license is None(알 수 없음)이고, Builder 는 그런 source 의 공개 게시를redistribution_unknown으로 거부한다.변경 내용
docs/policy/terms-matrix.md에 "Which datasets can be published" 절: 결정, 근거, 어디서 강제되는지, 카탈로그 전용 데이터셋이 게시 가능해지는 경로(spec 이전 시 조건 확인, chore: remaining migrate-to-spec backlog #409).tests/unit/test_declared_terms_decision.py: 조건이 있는 데이터셋은 모두 spec 기반 / 카탈로그 전용 데이터셋에는 조건이 없음 /to_dict()에서license: null/ 결정 문장이 문서에 있음.[Unreleased]Changed.완료 조건 2번 (Builder 의 거부 테스트)
이미 있다 — 새로 추가하지 않았다. Builder
main의service/redistribution.py가 선언 없는 데이터셋을unknown("the dataset declares no redistribution terms")으로 판정하고tests/unit/test_redistribution.py::test_unknown_terms_block_a_public_publish_only가 공개 게시 거부를 고정한다 (코드와 테스트 이름을 읽어 확인했고, 이 PR 에서 다시 돌리지는 않았다). 비공개 게시는 여전히 가능하다는 점도 문서에 적었다.검증
실행: 런타임 데이터셋 156개 중
license가 있는 것 25개, spec 이 있는 것 25개. 새 테스트 4개 통과, ruff 통과.review:R3이슈다 — 작성자가 아닌 사람의 승인이 필요하다. 결정 자체가 정책이므로 다르게 정하고 싶으면 이 PR 을 닫으면 된다.🤖 Generated with Claude Code