git 저장소가 없어 히스토리·협업 기반이 아예 없던 상태를 연다.
함께 문서를 재편했다. 그동안 문서가 있어도 "이 제품이 뭘 푸는가"와
"어떻게 도는가"를 담은 문서가 없어서, 목표 문장이 backend/frontend
README 두 곳에 복붙돼 있었다 — 상위 문서가 없어 아래로 샌 것이다.
신설
README.md 레포 진입점 + 문서 지도 + 문서 규칙 4가지
AGENTS.md 에이전트·신규 합류자용 함정 목록과 규약
(CLAUDE.md 는 여기로 걸린 심볼릭 링크)
docs/PRODUCT.md 제품 정의 — 문제·사용자·원칙·**non-goals**·성공 기준
docs/ARCHITECTURE.md payload 경계·발행 파이프라인·서빙 결정·앱 분리 설계
이동
backend/docs/DECISIONS.md → docs/DECISIONS.md
백엔드만의 결정이 아니다. 게다가 코드 주석 ~25곳이 이미
`docs/DECISIONS.md` 로 적고 있어 레포 루트 기준으로는 그게 맞다.
갱신
docs/DEPLOY.md 서빙 결정 반영 — nginx 정적 서빙이 지금 경로(3절),
Azure 는 나중에 켤 때(4절)로 분리
docs/ARCHITECTURE.md 사이트 = 한 장(2026-08-31) 구조 반영
docs/COLLECTION_SEO_AEO_FLOW.md
robots.txt·sitemap.xml 은 오리진 루트에만 굽는다는 점 명시
frontend/site/scripts/prerender.ts
헤더 주석의 렌더 보고서 경로가 실제(422줄)와 달라 수정
.gitignore
★ CLAUDE.md 를 더 이상 무시하지 않는다. 에이전트 지침은 팀과 모든
에이전트가 공유하는 규약이라 커밋해야 한다 — 무시하면 클론한 사람이
"배포 후 republish_all.py 필수" 같은 함정을 전달받지 못한다.
개인용 오버라이드는 ~/.claude/CLAUDE.md 에 둔다.
354 lines
18 KiB
Python
354 lines
18 KiB
Python
"""정적 빌드 + 발행 — 상호명에서 발행까지 한 바퀴.
|
|
|
|
★ 이 경로가 절대 하면 안 되는 것:
|
|
- 미검증 fact 를 실은 사이트를 발행하는 것
|
|
- 고유 콘텐츠 0건짜리 템플릿 페이지를 발행하는 것
|
|
- JSON-LD 와 화면이 다른 사이트를 발행하는 것
|
|
- 렌더 결과를 확인하지 않고 "발행됨" 으로 남기는 것 (페이지 없는 발행)
|
|
- 해지를 물리 삭제로 처리하는 것
|
|
|
|
★ 게이트는 이제 **렌더러가 실제로 구운 결과**를 보고 판정한다. 여기서는 그 보고서를
|
|
백엔드가 어떻게 처리하는지를 본다 — 구조화 데이터 ↔ 화면 값 대조 자체는
|
|
렌더러 쪽 테스트가 본다(frontend/site/src/seo/verify.test.ts).
|
|
"""
|
|
import uuid
|
|
|
|
from sqlalchemy import text
|
|
|
|
from common.enums import (
|
|
BuildStatus,
|
|
ErrorType,
|
|
FactStatus,
|
|
JobStatus,
|
|
MediaStatus,
|
|
PublishAction,
|
|
PublishRejectReason,
|
|
SiteStatus,
|
|
SourceType,
|
|
)
|
|
from crud.job_crud import JobQueue
|
|
from worker.handlers import build_handler
|
|
from worker.runner import Worker
|
|
|
|
REQUIRED = {
|
|
"check_in_time": "15:00", "check_out_time": "11:00",
|
|
"cancel_policy": "이용 7일 전 100% 환불, 3일 전 50% 환불",
|
|
"cooking_allowed": "true", "pet_allowed": "false",
|
|
}
|
|
|
|
|
|
async def _run():
|
|
assert await Worker("w", JobQueue(), build_handler(), job_deadline_sec=60).process_one() is True
|
|
|
|
|
|
async def _place(client, h, name="빌드펜션"):
|
|
pid = (await client.post("/v1/place", headers=h, json={"name": name, "category": 1})).json()["place"]["place_id"]
|
|
await client.post(f"/v1/place/{pid}/verify", headers=h, json={
|
|
"source": 2, "road_address": f"강원특별자치도 양양군 현북면 하조대3길 {uuid.uuid4().hex[:3]}",
|
|
"phone": "033-672-0000", "latitude": "38.0219217", "longitude": "128.7221449",
|
|
})
|
|
return pid
|
|
|
|
|
|
async def _verified_facts(client, h, pid, extra=None):
|
|
"""필수 항목을 넣고 전부 승인 상태로 만든다(사람이 직접 입력 = 즉시 노출값)."""
|
|
for k, v in {**REQUIRED, **(extra or {})}.items():
|
|
await client.post(f"/v1/place/{pid}/fact", headers=h, json={"key": k, "value": v})
|
|
|
|
|
|
async def _approved_media(db_engine, pid, alt="2층 목조 건물 외관"):
|
|
async with db_engine.begin() as c:
|
|
await c.execute(
|
|
text("INSERT INTO media (media_id, place_id, url, origin_url, source_type, status, alt_text, label, sort_order) "
|
|
"VALUES (:m,:p,:u,:u,:s,:st,:a,:l,0)"),
|
|
{"m": uuid.uuid4(), "p": uuid.UUID(pid), "u": f"https://cdn.test/{uuid.uuid4().hex[:6]}.jpg",
|
|
"s": SourceType.CRAWL.value, "st": MediaStatus.APPROVED.value, "a": alt, "l": "외관"},
|
|
)
|
|
|
|
|
|
async def test_full_build_and_publish(auth_headers, client, db_engine):
|
|
"""검증: 필수 fact + 승인 사진이 갖춰진 사업장을 빌드·발행한다.
|
|
기대결과: 게이트 통과 → BUILT → PUBLISHED. 버전과 발행 기록이 남는다."""
|
|
h = await auth_headers("u1")
|
|
pid = await _place(client, h)
|
|
await _verified_facts(client, h, pid, {"intro": "하조대 해변 도보 3분 거리의 펜션입니다."})
|
|
await _approved_media(db_engine, pid)
|
|
|
|
job_id = (await client.post(f"/v1/place/{pid}/site/build", headers=h, json={"publish": True})).json()["job_id"]
|
|
await _run()
|
|
|
|
job = (await client.get(f"/v1/job/{job_id}", headers=h)).json()["job"]
|
|
assert job["status"] == JobStatus.DONE.value, job.get("last_error")
|
|
r = job["result"]
|
|
assert r["gate"]["passed"] is True, r["gate"]
|
|
assert r["build_status"] == "BUILT"
|
|
assert r["published"] is True
|
|
assert r["unique_content_count"] > 0
|
|
# ★ 백엔드는 HTML 을 굽지 않는다. payload 를 쓰고, 렌더러가 굽은 결과를 받아 판정한다.
|
|
assert r["payload_path"], "렌더러에 넘길 payload 가 없다"
|
|
assert r["routes"] > 0, "렌더러가 페이지를 하나도 굽지 않았다"
|
|
|
|
site = (await client.get(f"/v1/place/{pid}/site", headers=h)).json()
|
|
assert site["site"]["status"] == SiteStatus.PUBLISHED.value
|
|
assert site["current_version"]["build_status"] == BuildStatus.BUILT.value
|
|
|
|
|
|
async def test_unverified_fact_blocks_publish(auth_headers, client, db_engine):
|
|
"""검증: 미검증 fact 가 섞인 채로 빌드한다.
|
|
|
|
기대결과: ★ 스냅샷에서 전부 걸러져 페이지에 남는 내용이 없다 — NO_UNIQUE_CONTENT 로 막힌다.
|
|
(필수 항목 누락은 2026-08-27 부터 막지 않는다. 그래서 여기서 걸리는 사유가 바뀌었다.)"""
|
|
h = await auth_headers("u1")
|
|
pid = await _place(client, h, "미검증펜션")
|
|
# 크롤링으로만 들어온 값 = 후보 상태
|
|
for k, v in REQUIRED.items():
|
|
await client.post(f"/v1/place/{pid}/fact", headers=h, json={
|
|
"key": k, "value": v, "source_type": SourceType.CRAWL.value, "source_url": "https://ota.test/1"})
|
|
|
|
job_id = (await client.post(f"/v1/place/{pid}/site/build", headers=h, json={"publish": True})).json()["job_id"]
|
|
await _run()
|
|
|
|
r = (await client.get(f"/v1/job/{job_id}", headers=h)).json()["job"]["result"]
|
|
assert r["gate"]["passed"] is False
|
|
assert r["gate"]["reason"] == PublishRejectReason.NO_UNIQUE_CONTENT.name
|
|
assert r["build_status"] == "FAILED"
|
|
assert r.get("published") is None
|
|
|
|
site = (await client.get(f"/v1/place/{pid}/site", headers=h)).json()
|
|
assert site["site"]["status"] == SiteStatus.DRAFT.value, "★ 게이트를 막았는데 발행됐다"
|
|
|
|
|
|
async def test_no_unique_content_blocks_publish(auth_headers, client):
|
|
"""검증: 필수 항목은 다 채웠지만 값이 전부 짧은 정형값이고 소개문·FAQ·사진이 없다.
|
|
기대결과: ★ NO_UNIQUE_CONTENT — 이 가게를 구분할 문장이 하나도 없으면
|
|
같은 템플릿 대량 생성으로 보인다(스팸 판정 대상)."""
|
|
h = await auth_headers("u1")
|
|
pid = await _place(client, h, "빈껍데기펜션")
|
|
# 취소 규정을 한 글자로 — 형식은 갖췄지만 이 가게만의 내용이 없다.
|
|
for k, v in {**REQUIRED, "cancel_policy": "X"}.items():
|
|
await client.post(f"/v1/place/{pid}/fact", headers=h, json={"key": k, "value": v})
|
|
|
|
job_id = (await client.post(f"/v1/place/{pid}/site/build", headers=h, json={"publish": True})).json()["job_id"]
|
|
await _run()
|
|
|
|
r = (await client.get(f"/v1/job/{job_id}", headers=h)).json()["job"]["result"]
|
|
assert r["gate"]["passed"] is False
|
|
assert r["gate"]["reason"] == PublishRejectReason.NO_UNIQUE_CONTENT.name
|
|
|
|
|
|
async def test_missing_required_field_warns_but_publishes(auth_headers, client, db_engine):
|
|
"""검증: 취소 규정 없이 빌드한다.
|
|
|
|
기대결과: ★ **발행된다.** 막아야 할 것은 "틀린 정보가 나가는 것"이지 "덜 찬 것"이 아니다
|
|
(2026-08-27 결정). 대신 무엇이 비었는지는 경고로 알려줘야 한다 — 사장님이 나중에
|
|
채우면 재빌드된다. 여기서 막으면 주소·전화가 확인된 쓸모 있는 페이지조차 못 낸다."""
|
|
h = await auth_headers("u1")
|
|
pid = await _place(client, h, "규정없는펜션")
|
|
partial = {k: v for k, v in REQUIRED.items() if k != "cancel_policy"}
|
|
for k, v in partial.items():
|
|
await client.post(f"/v1/place/{pid}/fact", headers=h, json={"key": k, "value": v})
|
|
await _approved_media(db_engine, pid)
|
|
|
|
job_id = (await client.post(f"/v1/place/{pid}/site/build", headers=h, json={"publish": True})).json()["job_id"]
|
|
await _run()
|
|
|
|
r = (await client.get(f"/v1/job/{job_id}", headers=h)).json()["job"]["result"]
|
|
assert r["gate"]["passed"] is True, "필수 항목 누락은 더 이상 발행을 막지 않는다"
|
|
assert r["gate"]["warning"] == PublishRejectReason.REQUIRED_FACT_MISSING.name
|
|
assert "cancel_policy" in r["gate"]["missing"], "무엇이 비었는지는 계속 알려줘야 한다"
|
|
assert r["build_status"] == "BUILT"
|
|
|
|
site = (await client.get(f"/v1/place/{pid}/site", headers=h)).json()
|
|
assert site["site"]["status"] == SiteStatus.PUBLISHED.value
|
|
|
|
|
|
async def test_rejected_build_is_logged_with_reason(auth_headers, client):
|
|
"""검증: 게이트가 막은 뒤 발행 기록.
|
|
기대결과: REJECTED 로 남고 사유가 detail 에 있다 — 운영자가 뭘 고칠지 안다."""
|
|
h = await auth_headers("u1")
|
|
pid = await _place(client, h, "기록펜션")
|
|
for k, v in {**REQUIRED, "cancel_policy": "X"}.items():
|
|
await client.post(f"/v1/place/{pid}/fact", headers=h, json={"key": k, "value": v})
|
|
await client.post(f"/v1/place/{pid}/site/build", headers=h, json={"publish": True})
|
|
await _run()
|
|
|
|
logs = (await client.get(f"/v1/place/{pid}/site/log/list", headers=h)).json()["logs"]
|
|
assert logs
|
|
assert logs[0]["reject_reason"] == PublishRejectReason.NO_UNIQUE_CONTENT.value
|
|
assert logs[0]["detail"]["reason"] == "NO_UNIQUE_CONTENT"
|
|
|
|
|
|
async def test_build_requires_verified_place(auth_headers, client):
|
|
"""검증: 동일 업소 검증 전에 빌드한다.
|
|
기대결과: PLACE_NOT_VERIFIED — 주소·좌표가 없으면 JSON-LD 가 성립하지 않는다."""
|
|
h = await auth_headers("u1")
|
|
pid = (await client.post("/v1/place", headers=h, json={"name": "미검증", "category": 1})).json()["place"]["place_id"]
|
|
r = await client.post(f"/v1/place/{pid}/site/build", headers=h, json={})
|
|
assert r.json()["result"]["code"] == ErrorType.PLACE_NOT_VERIFIED.value
|
|
|
|
|
|
async def test_needs_rebuild_after_content_change(auth_headers, client, db_engine):
|
|
"""검증: 발행 후 노출값을 바꾼다.
|
|
기대결과: ★ needs_rebuild=true — 이 사업장만 재빌드하면 된다(전체 재빌드 금지)."""
|
|
h = await auth_headers("u1")
|
|
pid = await _place(client, h, "재빌드펜션")
|
|
await _verified_facts(client, h, pid, {"intro": "조용한 펜션입니다."})
|
|
await _approved_media(db_engine, pid)
|
|
await client.post(f"/v1/place/{pid}/site/build", headers=h, json={"publish": True})
|
|
await _run()
|
|
|
|
assert (await client.get(f"/v1/place/{pid}/site", headers=h)).json()["needs_rebuild"] is False
|
|
|
|
await client.post(f"/v1/place/{pid}/fact", headers=h, json={"key": "check_in_time", "value": "16:00"})
|
|
assert (await client.get(f"/v1/place/{pid}/site", headers=h)).json()["needs_rebuild"] is True
|
|
|
|
|
|
async def test_suspend_is_state_transition_not_delete(auth_headers, client, db_engine):
|
|
"""검증: 발행된 사이트를 중지하고 다시 재개한다.
|
|
기대결과: ★ 상태만 바뀌고 버전·행은 그대로 — 해지를 삭제로 처리하지 않는다."""
|
|
h = await auth_headers("u1")
|
|
pid = await _place(client, h, "해지펜션")
|
|
await _verified_facts(client, h, pid, {"intro": "펜션 소개문입니다."})
|
|
await _approved_media(db_engine, pid)
|
|
await client.post(f"/v1/place/{pid}/site/build", headers=h, json={"publish": True})
|
|
await _run()
|
|
version_id = (await client.get(f"/v1/place/{pid}/site", headers=h)).json()["current_version"]["site_version_id"]
|
|
|
|
body = (await client.post(f"/v1/place/{pid}/site/status", headers=h,
|
|
json={"action": PublishAction.SUSPEND.value})).json()
|
|
assert body["site"]["status"] == SiteStatus.SUSPENDED.value
|
|
assert body["current_version"]["site_version_id"] == version_id, "★ 중지가 버전을 지웠다"
|
|
|
|
body = (await client.post(f"/v1/place/{pid}/site/status", headers=h,
|
|
json={"action": PublishAction.RESUME.value})).json()
|
|
assert body["site"]["status"] == SiteStatus.PUBLISHED.value
|
|
|
|
|
|
async def test_versions_accumulate(auth_headers, client, db_engine):
|
|
"""검증: 두 번 빌드한다.
|
|
기대결과: 버전이 1, 2 로 쌓인다 — 예전에 뭐가 나갔는지 추적할 수 있다."""
|
|
h = await auth_headers("u1")
|
|
pid = await _place(client, h, "버전펜션")
|
|
await _verified_facts(client, h, pid, {"intro": "펜션 소개문입니다."})
|
|
await _approved_media(db_engine, pid)
|
|
|
|
await client.post(f"/v1/place/{pid}/site/build", headers=h, json={"publish": True})
|
|
await _run()
|
|
await client.post(f"/v1/place/{pid}/site/build", headers=h, json={"publish": True})
|
|
await _run()
|
|
|
|
versions = (await client.get(f"/v1/place/{pid}/site/version/list", headers=h)).json()["versions"]
|
|
assert [v["version"] for v in versions] == [2, 1]
|
|
|
|
|
|
async def test_site_is_scoped_to_company(auth_headers, client, other_company_id):
|
|
"""검증: 다른 회사 계정으로 남의 사이트를 본다.
|
|
기대결과: PLACE_NOT_FOUND."""
|
|
h1 = await auth_headers("o1")
|
|
pid = await _place(client, h1, "스코프펜션")
|
|
h2 = await auth_headers("o2", other_company_id)
|
|
r = await client.get(f"/v1/place/{pid}/site", headers=h2)
|
|
assert r.json()["result"]["code"] == ErrorType.PLACE_NOT_FOUND.value
|
|
|
|
|
|
# ── 렌더 결과 기반 게이트 ────────────────────────────────────────────────────
|
|
async def _built_place(client, h, db_engine):
|
|
pid = await _place(client, h)
|
|
await _verified_facts(client, h, pid, {"intro": "하조대 해변 도보 3분 거리의 펜션입니다."})
|
|
await _approved_media(db_engine, pid)
|
|
return pid
|
|
|
|
|
|
async def _build(client, h, pid, publish=True):
|
|
job_id = (await client.post(f"/v1/place/{pid}/site/build", headers=h,
|
|
json={"publish": publish})).json()["job_id"]
|
|
await _run()
|
|
return (await client.get(f"/v1/job/{job_id}", headers=h)).json()["job"]["result"]
|
|
|
|
|
|
async def test_렌더가_실패하면_발행하지_않는다(auth_headers, client, db_engine, monkeypatch):
|
|
"""검증: 렌더러가 페이지를 굽지 못했다고 보고한다.
|
|
기대결과: ★ FAILED. 페이지가 없는데 "발행됨"으로 남으면 사장님은 404 를 눌러야 안다."""
|
|
from services import render_report
|
|
|
|
async def _broken(slug, site_version, timeout_sec, poll_sec=1.0):
|
|
return {"schemaVersion": 1, "slug": slug, "siteId": "", "placeId": "",
|
|
"siteVersion": site_version, "ok": False, "renderedAt": "2026-01-01T00:00:00Z",
|
|
"routes": 0, "bundle": "t.js", "uniqueContentCount": 5, "jsonld": [],
|
|
"mismatches": [], "error": "번들이 없다"}
|
|
|
|
h = await auth_headers("u1")
|
|
pid = await _built_place(client, h, db_engine)
|
|
monkeypatch.setattr(render_report, "wait_for", _broken)
|
|
|
|
r = await _build(client, h, pid)
|
|
assert r["build_status"] == "FAILED"
|
|
assert r.get("published") is not True
|
|
assert "번들이 없다" in r["error"]
|
|
|
|
site = (await client.get(f"/v1/place/{pid}/site", headers=h)).json()
|
|
assert site["site"]["status"] != SiteStatus.PUBLISHED.value
|
|
|
|
|
|
async def test_구조화데이터가_화면과_다르면_발행하지_않는다(auth_headers, client, db_engine, monkeypatch):
|
|
"""검증: 렌더러가 JSON-LD 불일치를 보고한다.
|
|
기대결과: ★ JSONLD_MISMATCH — 검색엔진에만 다른 말을 하는 페이지는 나가면 안 된다."""
|
|
from services import render_report
|
|
|
|
async def _mismatch(slug, site_version, timeout_sec, poll_sec=1.0):
|
|
return {"schemaVersion": 1, "slug": slug, "siteId": "", "placeId": "",
|
|
"siteVersion": site_version, "ok": False, "renderedAt": "2026-01-01T00:00:00Z",
|
|
"routes": 0, "bundle": "t.js", "uniqueContentCount": 5, "jsonld": [],
|
|
"mismatches": ["telephone: '02-0000-0000' 이 화면에 없다"],
|
|
"error": "구조화 데이터가 화면 값과 다릅니다(1건)"}
|
|
|
|
h = await auth_headers("u1")
|
|
pid = await _built_place(client, h, db_engine)
|
|
monkeypatch.setattr(render_report, "wait_for", _mismatch)
|
|
|
|
r = await _build(client, h, pid)
|
|
assert r["build_status"] == "FAILED"
|
|
# ★ 사유가 "렌더 실패"로 뭉뚱그려지면 운영자가 손댈 곳을 알 수 없다.
|
|
assert r["gate"]["reason"] == PublishRejectReason.JSONLD_MISMATCH.name
|
|
assert r["mismatches"]
|
|
|
|
|
|
async def test_렌더_결과가_안_오면_발행하지_않는다(auth_headers, client, db_engine, monkeypatch):
|
|
"""검증: 렌더러가 죽어 보고서가 오지 않는다(타임아웃).
|
|
기대결과: ★ FAILED. 확인하지 못한 것을 발행됨으로 남기지 않는다."""
|
|
from services import render_report
|
|
|
|
async def _timeout(slug, site_version, timeout_sec, poll_sec=1.0):
|
|
return None
|
|
|
|
h = await auth_headers("u1")
|
|
pid = await _built_place(client, h, db_engine)
|
|
monkeypatch.setattr(render_report, "wait_for", _timeout)
|
|
|
|
r = await _build(client, h, pid)
|
|
assert r["build_status"] == "FAILED"
|
|
assert r.get("published") is not True
|
|
assert "렌더 결과" in r["error"]
|
|
|
|
|
|
async def test_렌더_보고서의_값이_그대로_박제된다(auth_headers, client, db_engine):
|
|
"""검증: site_versions 에 남는 jsonld·고유콘텐츠 수의 출처.
|
|
기대결과: ★ 렌더러가 실제로 내보낸 값이다 — 백엔드가 따로 계산하지 않는다.
|
|
(따로 계산하던 시절엔 게이트가 통과시킨 근거와 실제 페이지가 어긋날 수 있었다.)"""
|
|
h = await auth_headers("u1")
|
|
pid = await _built_place(client, h, db_engine)
|
|
r = await _build(client, h, pid)
|
|
|
|
assert r["build_status"] == "BUILT"
|
|
|
|
async with db_engine.begin() as c:
|
|
row = (await c.execute(
|
|
text("SELECT jsonld, unique_content_count FROM site.site_versions WHERE site_version_id = :v"),
|
|
{"v": uuid.UUID(r["site_version_id"])},
|
|
)).mappings().first()
|
|
|
|
assert row["unique_content_count"] == r["unique_content_count"]
|
|
assert row["jsonld"], "렌더러가 내보낸 JSON-LD 가 박제되지 않았다"
|
|
# 렌더러 대역이 내보낸 노드가 그대로 들어와야 한다(백엔드가 다시 만들지 않는다).
|
|
assert row["jsonld"][0]["@type"] == "LodgingBusiness"
|