revert(cafe24): 일괄수정 기능 제거

사용자 결정에 따라 화면·라우트·로직·문서를 모두 제거했다.

제거 대상:
  routes_bulk.py            (파일 삭제)
  templates/cafe24/bulk.html (파일 삭제)
  _nav.html                 「일괄수정」 탭
  router.py                 import·include·설명
  store.py                  find_style_blocks / replace_first_style_block / 정규식
  tests                     <style> 블록 교체 테스트 6건
  docs                      2-3 절, 경로 표 3줄, 구조도, Phase 7 표기

DB 는 손대지 않았다. 일괄수정은 전용 스키마를 만들지 않았고, 기존 테이블에 남은
기록은 **실제로 상품에 적용된 변경의 이력**이다. 특히 그때 만들어진 BACKUP revision
은 일괄 적용 이전 내용을 되찾을 유일한 수단이라 지우면 복구가 불가능해진다.
감사로그(action='bulk_style')도 누가 언제 무엇을 바꿨는지 남기는 기록이라 보존한다.
정말 지워야 한다면 별도로 요청받아 진행한다.

부수 수정: 예약 시각 검증 테스트가 실행 시각의 초에 따라 실패할 수 있었다.
`datetime-local` 은 초를 버리므로 지금 이 분(分)을 고르면 최대 59초 과거가 되는데,
테스트가 now 의 초를 고정하지 않아 경계에서 흔들렸다. now 를 고정해 결정적으로 만들었다
(3회 반복 실행으로 확인).

검증: 유닛테스트 59개 통과, 라우트 13개, 템플릿 컴파일 확인, 모듈에 bulk 참조 0건.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
2026-08-14 16:14:40 +09:00
parent 6dae0e45c9
commit 8ec38db6d1
10 changed files with 15 additions and 517 deletions
-3
View File
@@ -10,7 +10,6 @@
라우트가 많아 기능별 파일로 나눈다(다른 모듈의 단일 router.py 패턴을 규모 때문에 라우트가 많아 기능별 파일로 나눈다(다른 모듈의 단일 router.py 패턴을 규모 때문에
확장한 것). 여기서는 루트 라우터를 만들고 서브 라우터를 결합한다. 확장한 것). 여기서는 루트 라우터를 만들고 서브 라우터를 결합한다.
routes_products 상품 목록/검색 · 상세설명 조회·편집·적용 routes_products 상품 목록/검색 · 상세설명 조회·편집·적용
routes_bulk 일괄수정 (검사 → 선택 적용)
routes_schedules 예약 등록·목록·취소 (실행은 worker.py) routes_schedules 예약 등록·목록·취소 (실행은 worker.py)
routes_system 연결(OAuth)·상태·API 로그·작업 로그 routes_system 연결(OAuth)·상태·API 로그·작업 로그
""" """
@@ -21,7 +20,6 @@ import logging
from fastapi import APIRouter from fastapi import APIRouter
from .routes_bulk import bulk_router
from .routes_products import products_router from .routes_products import products_router
from .routes_schedules import schedules_router from .routes_schedules import schedules_router
from .routes_system import system_router from .routes_system import system_router
@@ -31,7 +29,6 @@ logger = logging.getLogger("cafe24.router")
router = APIRouter(prefix="/cafe24", tags=["cafe24"]) router = APIRouter(prefix="/cafe24", tags=["cafe24"])
router.include_router(products_router) router.include_router(products_router)
router.include_router(bulk_router)
router.include_router(schedules_router) router.include_router(schedules_router)
router.include_router(system_router) router.include_router(system_router)
-195
View File
@@ -1,195 +0,0 @@
"""카페24 일괄수정 — 상세페이지 맨 위 `<style>` 블록 통일.
**미리보기 먼저, 적용은 확인 후.** 87개 상품을 한 번에 쓰는 작업이라, 무엇이
지워지는지 보지 않고 실행하면 남의 CSS(예: 비디오 반응형 스타일)가 조용히 사라진다.
그래서 두 단계로 나눈다.
1) 검사 GET /bulk/scan/{no} → 그 상품의 현재 <style> 내용과 교체 후 모양
2) 적용 POST /bulk/apply/{no} → 그 상품 하나만 실제로 반영
상품 1건당 1요청으로 쪼갠 이유:
- 87건을 한 요청으로 처리하면 1분 가까이 걸려 프록시 타임아웃에 걸린다.
- 브라우저가 순차 호출하며 진행률을 보여줄 수 있고, 한 건 실패가 나머지를 막지
않으며, 어디까지 됐는지 화면에 남는다.
적용 순서는 단건 편집(`routes_products.product_apply`)과 같은 원칙을 지킨다.
카페24 현재값 재조회 → BACKUP 버전 → 교체 → PUT → MANUAL 버전 + 감사로그
지문 대조는 하지 않는다 — 쓰기 직전에 방금 읽은 값을 그대로 쓰기 때문이다.
"""
from __future__ import annotations
import logging
from typing import Any
from fastapi import APIRouter, HTTPException, Request
from fastapi.responses import HTMLResponse
from app.integrations.cafe24 import Cafe24Error, build_cafe24_api, products
from . import store
from .common import base_ctx, guard, require_store
from .routes_products import _no_store
logger = logging.getLogger("cafe24.bulk")
bulk_router = APIRouter()
# 통일할 <style> 블록. 탭 들여쓰기까지 사용자가 준 모양 그대로 쓴다
# (format_html 은 <style> 안쪽을 건드리지 않으므로 그대로 저장된다).
TARGET_STYLE_BLOCK = "<style>\n\tdiv {\n\t\ttext-align: center;\n\t}\n</style>"
def _preview(html: str) -> dict[str, Any]:
"""현재 <style> 상태와 교체 후 모양을 요약한다."""
blocks = store.find_style_blocks(html or "")
current = blocks[0] if blocks else ""
after = store.replace_first_style_block(html or "", TARGET_STYLE_BLOCK)
return {
"style_count": len(blocks),
"current_style": current,
"already_target": current.strip() == TARGET_STYLE_BLOCK.strip(),
"will_change": after != (html or ""),
}
@bulk_router.get("/bulk", response_class=HTMLResponse)
def bulk_page(request: Request) -> HTMLResponse:
"""일괄수정 화면. 상품 목록만 서버에서 그리고, 검사는 브라우저가 순차 호출한다."""
from app.main import render_template # noqa: WPS433
checked = guard(request)
if not isinstance(checked, tuple):
return checked
st, user = checked
api = build_cafe24_api(st)
rows: list[dict[str, Any]] = []
error = ""
try:
raw_rows, _truncated = products.list_all_products(api.client)
st.upsert_products([products.normalize_product(r) for r in raw_rows])
rows = [
{
"product_no": normalized["product_no"],
"product_name": normalized["product_name"],
"display": normalized["display"],
"selling": normalized["selling"],
}
for normalized in (products.normalize_product(r) for r in raw_rows)
]
except Cafe24Error as exc:
error = str(exc)
logger.warning("카페24 상품 목록 조회 실패: %s", exc)
ctx = base_ctx(request, user, active_tab="bulk")
ctx.update(
{
"page_title": "카페24 — 일괄수정",
"page_subtitle": "상세페이지 <style> 블록 통일",
"rows": rows,
"total": len(rows),
"target_style": TARGET_STYLE_BLOCK,
"error": error,
}
)
return _no_store(render_template(request, "cafe24/bulk.html", ctx))
@bulk_router.get("/bulk/scan/{product_no}")
def bulk_scan(request: Request, product_no: int) -> dict[str, Any]:
"""상품 1건의 현재 <style> 상태(읽기 전용)."""
st, _user = require_store(request)
api = build_cafe24_api(st)
try:
desc = products.fetch_descriptions(api.client, product_no)
except Cafe24Error as exc:
raise HTTPException(status_code=502, detail=str(exc)) from None
pc = _preview(desc.description)
mobile = _preview(desc.mobile_description)
return {
"product_no": product_no,
"product_name": desc.product_name,
"separated_mobile": desc.separated_mobile,
"pc": pc,
# 분리 상품만 모바일을 따로 본다(미분리는 PC 값을 그대로 쓴다).
"mobile": mobile if desc.separated_mobile else None,
"will_change": pc["will_change"] or (desc.separated_mobile and mobile["will_change"]),
}
@bulk_router.post("/bulk/apply/{product_no}")
def bulk_apply(request: Request, product_no: int) -> dict[str, Any]:
"""상품 1건의 맨 위 <style> 을 통일된 블록으로 교체한다."""
st, user = require_store(request)
actor = str(user.get("email") or "")
api = build_cafe24_api(st)
# 1) 현재값을 다시 읽는다 — 로컬 값이나 방금 검사한 값을 믿지 않는다.
try:
current = products.fetch_descriptions(api.client, product_no)
except Cafe24Error as exc:
st.log_audit(
actor=actor, action="bulk_style", product_no=product_no,
result="FAIL", detail=f"현재값 조회 실패: {exc}",
)
raise HTTPException(status_code=502, detail=str(exc)) from None
# 2) BACKUP — 유일한 복구 수단
backup_id = st.add_revision(
product_no=product_no,
html_content=current.description,
revision_type=store.REVISION_BACKUP,
memo="일괄 style 교체 직전 자동 백업",
created_by=actor,
)
new_pc = store.format_html(
store.replace_first_style_block(current.description, TARGET_STYLE_BLOCK)
)
if current.separated_mobile:
new_mobile: str | None = store.format_html(
store.replace_first_style_block(current.mobile_description, TARGET_STYLE_BLOCK)
)
if new_mobile == current.mobile_description:
new_mobile = None
else:
new_mobile = new_pc
if new_pc == current.description and (
new_mobile is None or new_mobile == current.mobile_description
):
return {"product_no": product_no, "result": "SKIPPED", "detail": "이미 같은 내용", "backup_id": backup_id}
try:
products.update_descriptions(
api.client, product_no, description=new_pc, mobile_description=new_mobile
)
except Cafe24Error as exc:
st.log_audit(
actor=actor, action="bulk_style", product_no=product_no,
revision_id=backup_id, result="FAIL", detail=str(exc),
)
logger.warning("카페24 상품 %s 일괄 style 적용 실패: %s", product_no, exc)
raise HTTPException(status_code=502, detail=f"{exc} (직전 내용은 버전 {backup_id})") from None
revision_id = st.add_revision(
product_no=product_no,
html_content=new_pc,
revision_type=store.REVISION_MANUAL,
memo="일괄 style 교체",
created_by=actor,
)
st.log_audit(
actor=actor, action="bulk_style", product_no=product_no,
revision_id=revision_id, result="SUCCESS",
detail=f"style 통일 (백업 {backup_id}"
+ (", 모바일 동시 반영)" if new_mobile is not None else ")"),
)
return {
"product_no": product_no,
"result": "SUCCESS",
"backup_id": backup_id,
"revision_id": revision_id,
}
-28
View File
@@ -377,34 +377,6 @@ def _collapse_short_blocks(lines: list[str]) -> list[str]:
return out return out
# ════════════════════════════════════════════════════════════
# <style> 블록 일괄 교체
#
# 상세페이지 맨 위의 <style> 을 정해진 내용으로 통일할 때 쓴다.
# **맨 앞 블록 하나만** 건드린다. 아래쪽에 다른 <style> 이 더 있으면 그건 그대로
# 두고, 화면에서 "블록 2개" 로 알려 사람이 판단하게 한다 — 일괄 작업이 남의 CSS 를
# 조용히 지우는 것이 가장 위험하다.
# ════════════════════════════════════════════════════════════
_STYLE_BLOCK_RE = re.compile(r"<style\b[^>]*>.*?</style>", re.IGNORECASE | re.DOTALL)
def find_style_blocks(html: str) -> list[str]:
"""상세설명 안의 <style>…</style> 블록 전체(원문 그대로)."""
return _STYLE_BLOCK_RE.findall(html or "")
def replace_first_style_block(html: str, new_block: str) -> str:
"""맨 앞 <style> 블록을 new_block 으로 교체한다.
블록이 하나도 없으면 맨 앞에 넣는다. 두 번째 이후 블록은 건드리지 않는다.
"""
source = html or ""
match = _STYLE_BLOCK_RE.search(source)
if match is None:
return new_block + ("\n" + source.lstrip("\n") if source.strip() else "")
return source[: match.start()] + new_block + source[match.end() :]
# ════════════════════════════════════════════════════════════ # ════════════════════════════════════════════════════════════
# 예약 입력 검증 # 예약 입력 검증
# #
@@ -1,9 +1,7 @@
{# 카페24 모듈 공용 상단 탭. active_tab: products | bulk | schedules | system #} {# 카페24 모듈 공용 상단 탭. active_tab: products | schedules | system #}
<div class="erp-page-actions" style="display:flex;gap:8px;flex-wrap:wrap;align-items:center;"> <div class="erp-page-actions" style="display:flex;gap:8px;flex-wrap:wrap;align-items:center;">
<a class="erp-btn {% if active_tab=='products' %}erp-btn-primary{% else %}erp-btn-outline{% endif %}" <a class="erp-btn {% if active_tab=='products' %}erp-btn-primary{% else %}erp-btn-outline{% endif %}"
href="/cafe24/">상품관리</a> href="/cafe24/">상품관리</a>
<a class="erp-btn {% if active_tab=='bulk' %}erp-btn-primary{% else %}erp-btn-outline{% endif %}"
href="/cafe24/bulk">일괄수정</a>
<a class="erp-btn {% if active_tab=='schedules' %}erp-btn-primary{% else %}erp-btn-outline{% endif %}" <a class="erp-btn {% if active_tab=='schedules' %}erp-btn-primary{% else %}erp-btn-outline{% endif %}"
href="/cafe24/schedules">예약관리</a> href="/cafe24/schedules">예약관리</a>
<a class="erp-btn {% if active_tab=='system' %}erp-btn-primary{% else %}erp-btn-outline{% endif %}" <a class="erp-btn {% if active_tab=='system' %}erp-btn-primary{% else %}erp-btn-outline{% endif %}"
@@ -1,192 +0,0 @@
{% extends "erp_base.html" %}
{% block head_extra %}
<link rel="stylesheet" href="/static/cafe24.css?v=20260814p" />
{% endblock %}
{% block content %}
{% include "cafe24/_nav.html" %}
{% if error %}
<div class="cf24-flash cf24-flash-err">
카페24 조회에 실패했습니다: {{ error }}<br />
<a href="/cafe24/system">시스템 화면에서 연결 상태를 확인하세요.</a>
</div>
{% endif %}
<div class="erp-card cf24-card">
<div class="cf24-card-head">
<h3>상세페이지 &lt;style&gt; 블록 통일</h3>
<span class="cf24-muted">전체 {{ total }}건</span>
</div>
<p class="cf24-note">
각 상품 상세설명의 <strong>맨 위 <code>&lt;style&gt;</code> 블록 하나</strong>를 아래 내용으로 바꿉니다.
아래쪽에 <code>&lt;style&gt;</code> 이 더 있으면 그건 건드리지 않고 「블록 2개」로 표시합니다.<br />
<strong class="cf24-warn">지금 들어 있는 CSS 는 사라집니다.</strong>
비디오 반응형처럼 필요한 규칙이 있을 수 있으니, <strong>먼저 검사</strong>해서 무엇이 지워지는지
확인하고 상품을 골라 적용하세요. 적용 직전 내용은 상품별 <code>BACKUP</code> 버전으로 보관됩니다.
PC/모바일 분리 상품은 모바일도 함께 바꿉니다.
</p>
<label class="cf24-label" for="cf24-target">교체할 내용</label>
<textarea id="cf24-target" class="cf24-html" rows="6" readonly>{{ target_style }}</textarea>
<div class="cf24-toolbar" style="margin-top:12px;">
<button class="erp-btn erp-btn-primary" type="button" id="cf24-scan">1. 전체 검사</button>
<button class="erp-btn erp-btn-outline" type="button" id="cf24-apply" disabled>2. 선택한 상품에 적용</button>
<span class="cf24-muted" id="cf24-progress">검사를 먼저 실행하세요.</span>
</div>
</div>
<div class="erp-card cf24-card">
<div class="cf24-card-head">
<h3>검사 결과</h3>
<span class="cf24-muted">변경이 필요한 상품만 자동 선택됩니다</span>
</div>
<div class="cf24-scroll">
<table class="erp-table cf24-compact" id="cf24-bulk-table">
<thead>
<tr>
<th style="width:34px;"><input type="checkbox" id="cf24-check-all" /></th>
<th style="width:60px;">번호</th>
<th>상품명</th>
<th style="width:70px;">style</th>
<th>현재 내용 (지워질 부분)</th>
<th style="width:90px;">상태</th>
</tr>
</thead>
<tbody>
{% for r in rows %}
<tr data-no="{{ r.product_no }}">
<td><input type="checkbox" class="cf24-pick" disabled /></td>
<td class="cf24-nowrap">{{ r.product_no }}</td>
<td>{{ r.product_name }}</td>
<td class="cf24-nowrap cf24-cell-count"></td>
<td class="cf24-cell-current"><span class="cf24-muted">검사 전</span></td>
<td class="cf24-nowrap cf24-cell-state"><span class="cf24-muted">대기</span></td>
</tr>
{% endfor %}
</tbody>
</table>
</div>
</div>
{% endblock %}
{% block scripts %}
<script>
(function () {
var rows = Array.prototype.slice.call(document.querySelectorAll("#cf24-bulk-table tbody tr"));
var scanBtn = document.getElementById("cf24-scan");
var applyBtn = document.getElementById("cf24-apply");
var progress = document.getElementById("cf24-progress");
var checkAll = document.getElementById("cf24-check-all");
function cell(tr, name) { return tr.querySelector(".cf24-cell-" + name); }
function setState(tr, text, cls) {
cell(tr, "state").innerHTML = '<span class="' + (cls || "cf24-muted") + '">' + text + "</span>";
}
// 한 건씩 순차 호출한다 — 87건을 한 요청으로 묶으면 프록시 타임아웃에 걸리고,
// 동시에 던지면 카페24 호출 제한(429)에 걸린다.
function walk(list, step, done) {
var i = 0;
(function next() {
if (i >= list.length) { done(); return; }
var tr = list[i++];
step(tr, function () {
progress.textContent = i + " / " + list.length;
next();
});
})();
}
scanBtn.addEventListener("click", function () {
scanBtn.disabled = true;
applyBtn.disabled = true;
progress.textContent = "검사 중… 0 / " + rows.length;
walk(rows, function (tr, next) {
setState(tr, "검사 중…");
fetch("/cafe24/bulk/scan/" + tr.dataset.no, { credentials: "same-origin", cache: "no-store" })
.then(function (res) { return res.ok ? res.json() : Promise.reject(res); })
.then(function (data) {
var pick = tr.querySelector(".cf24-pick");
cell(tr, "count").textContent = data.pc.style_count + "개";
if (data.pc.style_count > 1) {
cell(tr, "count").innerHTML += ' <span class="cf24-warn">주의</span>';
}
var current = data.pc.current_style || "";
cell(tr, "current").innerHTML = current
? "<code>" + current.replace(/[&<>]/g, function (c) {
return c === "&" ? "&amp;" : c === "<" ? "&lt;" : "&gt;";
}).slice(0, 400) + "</code>"
: '<span class="cf24-muted">없음 (새로 넣습니다)</span>';
pick.disabled = !data.will_change;
pick.checked = data.will_change;
if (data.pc.already_target && !data.will_change) setState(tr, "이미 동일", "cf24-muted");
else setState(tr, "변경 필요", "cf24-warn");
if (data.separated_mobile) {
cell(tr, "count").innerHTML += ' <span class="cf24-muted">/ 모바일분리</span>';
}
next();
})
.catch(function (res) {
setState(tr, "검사 실패", "cf24-err");
if (res && res.status === 401) window.location.href = "/login";
next();
});
}, function () {
scanBtn.disabled = false;
applyBtn.disabled = false;
var picked = rows.filter(function (tr) { return tr.querySelector(".cf24-pick").checked; }).length;
progress.textContent = "검사 완료 — 변경 필요 " + picked + "건 선택됨";
});
});
checkAll.addEventListener("change", function () {
rows.forEach(function (tr) {
var pick = tr.querySelector(".cf24-pick");
if (!pick.disabled) pick.checked = checkAll.checked;
});
});
applyBtn.addEventListener("click", function () {
var picked = rows.filter(function (tr) { return tr.querySelector(".cf24-pick").checked; });
if (!picked.length) { window.alert("선택된 상품이 없습니다."); return; }
if (!window.confirm(picked.length + "개 상품의 상세페이지에 바로 반영됩니다.\n\n" +
"지금 들어 있는 <style> 내용은 사라집니다(상품별 BACKUP 버전으로 보관).\n계속할까요?")) return;
scanBtn.disabled = true;
applyBtn.disabled = true;
progress.textContent = "적용 중… 0 / " + picked.length;
var ok = 0, fail = 0, skip = 0;
walk(picked, function (tr, next) {
setState(tr, "적용 중…");
fetch("/cafe24/bulk/apply/" + tr.dataset.no, { method: "POST", credentials: "same-origin" })
.then(function (res) {
return res.json().then(function (body) {
if (!res.ok) return Promise.reject(body);
return body;
});
})
.then(function (data) {
if (data.result === "SKIPPED") { skip++; setState(tr, "변경 없음", "cf24-muted"); }
else { ok++; setState(tr, "완료", "cf24-badge-ok"); tr.querySelector(".cf24-pick").checked = false; }
next();
})
.catch(function (body) {
fail++;
setState(tr, "실패", "cf24-err");
if (body && body.detail) cell(tr, "current").innerHTML =
'<span class="cf24-err">' + String(body.detail).slice(0, 200) + "</span>";
next();
});
}, function () {
scanBtn.disabled = false;
applyBtn.disabled = false;
progress.textContent = "적용 완료 — 성공 " + ok + " / 변경없음 " + skip + " / 실패 " + fail;
});
});
})();
</script>
{% endblock %}
@@ -1,7 +1,7 @@
{% extends "erp_base.html" %} {% extends "erp_base.html" %}
{% block head_extra %} {% block head_extra %}
<link rel="stylesheet" href="/static/cafe24.css?v=20260814p" /> <link rel="stylesheet" href="/static/cafe24.css?v=20260814q" />
{% endblock %} {% endblock %}
{% block content %} {% block content %}
@@ -1,7 +1,7 @@
{% extends "erp_base.html" %} {% extends "erp_base.html" %}
{% block head_extra %} {% block head_extra %}
<link rel="stylesheet" href="/static/cafe24.css?v=20260814p" /> <link rel="stylesheet" href="/static/cafe24.css?v=20260814q" />
{% endblock %} {% endblock %}
{% block content %} {% block content %}
@@ -1,7 +1,7 @@
{% extends "erp_base.html" %} {% extends "erp_base.html" %}
{% block head_extra %} {% block head_extra %}
<link rel="stylesheet" href="/static/cafe24.css?v=20260814p" /> <link rel="stylesheet" href="/static/cafe24.css?v=20260814q" />
{% endblock %} {% endblock %}
{% block content %} {% block content %}
+8 -63
View File
@@ -539,65 +539,6 @@ def test_format_then_encode_roundtrip():
assert saved == raw assert saved == raw
# ════════════════════════════════════════════════════════════
# <style> 블록 일괄 교체
# ════════════════════════════════════════════════════════════
_NEW_STYLE = "<style>\n\tdiv {\n\t\ttext-align: center;\n\t}\n</style>"
def test_find_style_blocks():
html = "<style>a{}</style><div>x</div><style type=\"text/css\">b{}</style>"
blocks = store.find_style_blocks(html)
assert len(blocks) == 2
assert blocks[0] == "<style>a{}</style>"
def test_replace_first_style_block_only():
"""두 번째 이후 <style> 은 건드리지 않는다(남의 CSS 를 조용히 지우지 않게)."""
html = "<style>OLD</style><div>x</div><style>KEEP</style>"
out = store.replace_first_style_block(html, _NEW_STYLE)
assert "OLD" not in out
assert "<style>KEEP</style>" in out
assert out.startswith(_NEW_STYLE)
assert "<div>x</div>" in out
def test_replace_style_inserts_when_missing():
out = store.replace_first_style_block("<div>x</div>", _NEW_STYLE)
assert out.startswith(_NEW_STYLE)
assert "<div>x</div>" in out
def test_replace_style_is_idempotent():
once = store.replace_first_style_block("<style>OLD</style><div>x</div>", _NEW_STYLE)
assert store.replace_first_style_block(once, _NEW_STYLE) == once
def test_replace_style_survives_formatting():
"""정리(포맷)를 거쳐도 <style> 안 탭·줄바꿈이 그대로 남는다."""
formatted = store.format_html(
store.replace_first_style_block("<style>OLD</style><div>x</div>", _NEW_STYLE)
)
assert "\tdiv {" in formatted
assert "\t\ttext-align: center;" in formatted
assert len(store.find_style_blocks(formatted)) == 1
def test_replace_style_keeps_multiline_original_shape():
"""원문이 여러 줄이어도 앞 블록만 정확히 잘라낸다."""
html = '<style>\n/* 비디오 반응형 */\n.video{max-width:100%}\n</style>\n<img src="a.gif">'
out = store.replace_first_style_block(html, _NEW_STYLE)
assert "비디오" not in out
assert '<img src="a.gif">' in out
def test_fingerprint_detects_change():
a = store.fingerprint("<p>A</p>")
assert a == store.fingerprint("<p>A</p>")
assert a != store.fingerprint("<p>B</p>")
assert len(a) == 32
# ════════════════════════════════════════════════════════════ # ════════════════════════════════════════════════════════════
# 예약 — 입력 검증 # 예약 — 입력 검증
# ════════════════════════════════════════════════════════════ # ════════════════════════════════════════════════════════════
@@ -628,10 +569,14 @@ def test_parse_schedule_at_rejects_past_and_garbage():
def test_parse_schedule_at_allows_one_minute_grace(): def test_parse_schedule_at_allows_one_minute_grace():
"""폼을 채우는 동안 시간이 흐른 경우를 거부하지 않는다.""" """`datetime-local` 은 초를 버린다 — 지금 이 분(分)을 고른 것을 거부하면 안 된다.
base = now_kst()
just_now = (base - timedelta(seconds=30)).strftime("%Y-%m-%dT%H:%M") now 의 초를 고정해 실행 시각에 따라 결과가 달라지지 않게 한다(예전에 이 테스트가
assert store.parse_schedule_at(just_now, now=base) is not None 초에 따라 실패했다).
"""
base = now_kst().replace(second=40, microsecond=0)
this_minute = base.strftime("%Y-%m-%dT%H:%M") # 초가 잘려 base 보다 40초 과거
assert store.parse_schedule_at(this_minute, now=base) is not None
def test_describe_schedule_action(): def test_describe_schedule_action():
+3 -30
View File
@@ -26,7 +26,6 @@ app/integrations/cafe24/ ← 공통 (상품관리 + 향후 주문관리
app/modules/cafe24/ ← 상품관리 모듈 app/modules/cafe24/ ← 상품관리 모듈
├─ router.py 루트 라우터(prefix=/cafe24) + 서브 라우터 결합 ├─ router.py 루트 라우터(prefix=/cafe24) + 서브 라우터 결합
├─ routes_products.py 2분할 화면 · 편집기 조각 · 적용(쓰기) ├─ routes_products.py 2분할 화면 · 편집기 조각 · 적용(쓰기)
├─ routes_bulk.py 일괄수정 (검사 → 선택 적용)
├─ routes_schedules.py 예약 등록·목록·취소 ├─ routes_schedules.py 예약 등록·목록·취소
├─ worker.py 예약 실행기 (compose 서비스 dbx-cafe24-worker) ├─ worker.py 예약 실행기 (compose 서비스 dbx-cafe24-worker)
├─ routes_system.py 연결(OAuth)·상태·API 로그·작업 로그 ├─ routes_system.py 연결(OAuth)·상태·API 로그·작업 로그
@@ -35,7 +34,7 @@ app/modules/cafe24/ ← 상품관리 모듈
├─ store.py 순수 로직 — 상수·상태전이·재시도 규칙·검증 ├─ store.py 순수 로직 — 상수·상태전이·재시도 규칙·검증
├─ tests/ DB/네트워크 없는 유닛테스트 ├─ tests/ DB/네트워크 없는 유닛테스트
└─ templates/cafe24/ _nav.html · products.html(2분할) · └─ templates/cafe24/ _nav.html · products.html(2분할) ·
_editor.html(오른쪽 조각) · bulk.html · _editor.html(오른쪽 조각) ·
schedules.html · system.html schedules.html · system.html
``` ```
@@ -56,9 +55,6 @@ app/modules/cafe24/ ← 상품관리 모듈
| `GET /cafe24/products/{product_no}/pane` | 오른쪽 편집기 조각 (JS 가 가져감) | `cafe24` | | `GET /cafe24/products/{product_no}/pane` | 오른쪽 편집기 조각 (JS 가 가져감) | `cafe24` |
| `GET /cafe24/products/{product_no}` | → `/cafe24/?selected=` 로 리다이렉트(옛 주소) | `cafe24` | | `GET /cafe24/products/{product_no}` | → `/cafe24/?selected=` 로 리다이렉트(옛 주소) | `cafe24` |
| `POST /cafe24/products/{product_no}/apply` | 편집한 HTML 을 카페24에 즉시 적용 | `cafe24` | | `POST /cafe24/products/{product_no}/apply` | 편집한 HTML 을 카페24에 즉시 적용 | `cafe24` |
| `GET /cafe24/bulk` | 일괄수정 화면 (`<style>` 통일) | `cafe24` |
| `GET /cafe24/bulk/scan/{product_no}` | 상품 1건 검사 (JSON, 읽기 전용) | `cafe24` |
| `POST /cafe24/bulk/apply/{product_no}` | 상품 1건 적용 (JSON) | `cafe24` |
| `GET /cafe24/schedules` | 예약 목록 · 취소 | `cafe24` | | `GET /cafe24/schedules` | 예약 목록 · 취소 | `cafe24` |
| `POST /cafe24/schedules` | 예약 등록 (편집기에서) | `cafe24` | | `POST /cafe24/schedules` | 예약 등록 (편집기에서) | `cafe24` |
| `POST /cafe24/schedules/{id}/cancel` | 대기 중 예약 취소 | `cafe24` | | `POST /cafe24/schedules/{id}/cancel` | 대기 중 예약 취소 | `cafe24` |
@@ -153,29 +149,7 @@ app/modules/cafe24/ ← 상품관리 모듈
--- ---
### 2-3. 일괄수정 (`<style>` 블록 통일) ### 2-3. 예약관리 (지정 시각 자동 적용)
87개 상품에 한 번에 쓰는 작업이라 **검사 → 선택 → 적용** 2단계로 나눈다. 무엇이
지워지는지 보지 않고 실행하면 남의 CSS(예: 비디오 반응형 스타일)가 조용히 사라진다.
- **맨 앞 `<style>` 블록 하나만** 바꾼다(`store.replace_first_style_block`). 아래쪽에
`<style>` 이 더 있으면 건드리지 않고 화면에 「블록 2개 · 주의」로 표시한다.
블록이 없으면 맨 앞에 넣는다.
- 검사 결과 표에 **지금 들어 있는 CSS 를 그대로 보여준다** — 그게 지워질 내용이다.
변경이 필요한 상품만 자동 선택되고, 이미 같은 내용이면 「이미 동일」로 제외된다.
- 적용은 상품 1건당 1요청이다. 87건을 한 요청으로 묶으면 1분 가까이 걸려 프록시
타임아웃에 걸리고, 동시에 던지면 호출 제한(429)에 걸린다. 브라우저가 순차 호출하며
진행률을 보여주고, 한 건 실패가 나머지를 막지 않는다.
- 적용 순서는 단건 편집과 같다: **현재값 재조회 → BACKUP → 교체 → PUT → MANUAL +
감사로그**(`action=bulk_style`). 지문 대조는 없다 — 방금 읽은 값을 바로 쓰기 때문.
- PC/모바일 분리 상품은 각 필드의 `<style>` 블록을 각각 바꾼다.
교체할 내용은 `routes_bulk.TARGET_STYLE_BLOCK` 에 있다. `<style>` 안쪽은
`format_html` 이 건드리지 않으므로 탭 들여쓰기까지 그대로 저장된다.
---
### 2-4. 예약관리 (지정 시각 자동 적용)
**되돌리기(자동 복원)는 쓰지 않는다.** 예약은 "그 시각에 이 내용을 적용" 하나뿐이다. **되돌리기(자동 복원)는 쓰지 않는다.** 예약은 "그 시각에 이 내용을 적용" 하나뿐이다.
한 예약에서 세 가지를 각각 고를 수 있고, 하나 이상은 반드시 골라야 한다(DB CHECK 제약). 한 예약에서 세 가지를 각각 고를 수 있고, 하나 이상은 반드시 골라야 한다(DB CHECK 제약).
@@ -299,7 +273,6 @@ cafe24_oauth_tokens 저장
모바일 소스를 따로 보여주지 않는다 — 한쪽만 바뀌어 어긋나는 사고가 없어진다. 모바일 소스를 따로 보여주지 않는다 — 한쪽만 바뀌어 어긋나는 사고가 없어진다.
분리 사용 상품의 모바일 내용이 PC 와 달랐다면 덮어쓰기 전에 **그 내용도 BACKUP 분리 사용 상품의 모바일 내용이 PC 와 달랐다면 덮어쓰기 전에 **그 내용도 BACKUP
revision 으로 남긴다**(백업이 없으면 되찾을 방법이 없다). revision 으로 남긴다**(백업이 없으면 되찾을 방법이 없다).
일괄수정(`routes_bulk`)은 각 필드의 `<style>` 블록만 바꾼다.
- 실패해도 BACKUP 은 이미 남아 있으므로 오류 메시지에 버전 번호를 알려준다. - 실패해도 BACKUP 은 이미 남아 있으므로 오류 메시지에 버전 번호를 알려준다.
--- ---
@@ -390,7 +363,7 @@ DB·네트워크 없이 암호화 왕복, 토큰 만료/자동갱신, 상태 노
| 4 | 즉시 적용 · BACKUP · Revision · 감사로그 | ✅ 완료 | | 4 | 즉시 적용 · BACKUP · Revision · 감사로그 | ✅ 완료 |
| 5 | 예약 DB · Worker(compose 서비스) · 예약관리 화면 | ✅ 완료 | | 5 | 예약 DB · Worker(compose 서비스) · 예약관리 화면 | ✅ 완료 |
| 6 | 자동 종료/복원 · 롤백 | ✖ 되돌리기는 쓰지 않기로 결정. 버전 선택 복원만 남음 | | 6 | 자동 종료/복원 · 롤백 | ✖ 되돌리기는 쓰지 않기로 결정. 버전 선택 복원만 남음 |
| 7 | 일괄 수정 · 일괄 예약 · Rate limit 제어 | ◐ `<style>` 통일 일괄수정 완료. 일괄 예약 예정 | | 7 | 일괄 수정 · 일괄 예약 · Rate limit 제어 | ✖ 일괄수정은 사용하지 않기로 제거. 필요해지면 다시 논의 |
Phase 5 의 worker 는 `app/modules/cafe24/worker.py` 에 둔다 — `Dockerfile` 이 Phase 5 의 worker 는 `app/modules/cafe24/worker.py` 에 둔다 — `Dockerfile` 이
`COPY app/ ./app/` 만 하므로 `scripts/` 에 두면 이미지에 포함되지 않는다. `COPY app/ ./app/` 만 하므로 `scripts/` 에 두면 이미지에 포함되지 않는다.