ワーカーの段を、画面から入れ替えられるようにする

段の並びは「詰まったら次へ」の順そのものなのに、変えるには相手を選び直すしか
なかった。3 段あれば 3 つとも選び直すことになり、そのあいだに 1 つ間違えると
無人で回る層が別の相手に回り続ける。

↑↓ はワーカーのフォームの submit として出す。form は入れ子にできないので、
中にもう 1 枚置くと内側が丸ごと無視される(押しても何も起きないボタンになる)。
同じフォームなら、書きかけの欄も一緒に保存してから動く —— 動かすために
保存し直す手間も要らない。

端では出さない(押せないボタンを置かない)。まだ相手を選んでいない末尾の行にも
出さない —— 動かす中身が無い。押された形だけが入ってきたときは何もしない。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Author: claude

CLAUDE.md

@@ -910,7 +910,13 @@ GeoNames 全世界地名辞典 = `geonames`(いずれも 348 言語版・195 か
       **ワーカーも同じ面に置く**(`app/views/ai_workers.py`)—— 何を回すかと、それを
       誰に回すかは 1 つの話で、離すと「なぜこの相手に回ったのか」を別の面と
       突き合わせて読むことになる
-    - **ワーカーは相手のセレクトに並ぶ**(`workers.OPTION_PREFIX`)。欄を分けていた
+    - **段は画面から入れ替えられる**(`views/ai_workers.py` の `_move_step_html`)。
+    並びは「詰まったら次へ」の順そのものなのに、変えるには相手を選び直すしか
+    なかった —— 3 段あれば 3 つとも選び直すことになり、そのあいだに 1 つ間違えると
+    無人で回る層が別の相手に回り続ける。**↑↓ はワーカーのフォームの submit にする**
+    (`<form>` は入れ子にできないので、中にもう 1 枚置くと内側が丸ごと無視される)
+    —— 同じフォームなら**書きかけの欄も一緒に保存してから動く**。端では出さない
+  - **ワーカーは相手のセレクトに並ぶ**(`workers.OPTION_PREFIX`)。欄を分けていた
       頃は相手とワーカーの両方を選べて、**どちらが効くのかが画面から読めなかった**
       (効くのはワーカー)。1 つの欄にすれば選べるのは片方だけになる。
       **ワーカーを選んだ回にはモデルも考える量も出さない**(保存時にも落とす)——

app/pages.py

@@ -219,6 +219,9 @@ def doc_url(source: str, doc_id: int) -> str:
      (table.ai-settings に掛けているのと同じ調整) */
   details.prompt-open { margin-top: 0.2rem; }
   details.prompt-open > summary { font-weight: normal; padding: 0.1rem 0; }
+  /* ワーカーの段を 1 つ上下へ動かす印(`views/ai_workers.py`)。**行から浮かせない**
+     —— 段の並びは「詰まったら次へ」の順そのもので、相手の名前の隣が読む場所 */
+  .step-move { font-size: .8rem; padding: .05rem .35rem; margin-left: .2rem; }
   .muted { color: #666; font-size: 0.85rem; }
   .pager { display: flex; gap: 1rem; align-items: baseline; margin-top: 1rem; }
   /* 玄関の使用量の表に添える 1 行（取り直す口と、全部の相手への入口）。

app/views/ai_workers.py

@@ -42,7 +42,9 @@ def _percent(provider: str) -> str:
     return f'<span class="muted">{busiest:.0f}% 使用{mark}</span>'
 
 
-def _step_row(index: int, step: workers.Step | None, backend_select, model_select) -> str:
+def _step_row(
+    index: int, step: workers.Step | None, backend_select, model_select, last: int = -1
+) -> str:
     """段 1 つぶんの欄。
 
     **考える量の欄は持たない。** 考える量はモデルの名前に畳んである
@@ -59,20 +61,53 @@ def _step_row(index: int, step: workers.Step | None, backend_select, model_selec
         '<div class="sweep-row">'
         f'<p><label>{index + 1} 番目<br>'
         f'{backend_select(current, "step_backend", empty_label=_EMPTY_STEP)}</label>'
-        f" {picked}</p>"
+        f" {picked} {_move_step_html(index, step, last)}</p>"
         f'<p><label>モデル<br>{model_select(current, step.model if step else "", "step_model")}'
         "</label></p>"
         "</div>"
     )
 
 
+def _move_step_html(index: int, step: workers.Step | None, last: int) -> str:
+    """段を 1 つ上/下へ動かす印。**段の並びは「詰まったら次へ」の順そのもの**。
+
+    **入れ替えるのに打ち直させない。** 並びを変えるには相手を選び直すしかなく、
+    3 段あれば 3 つとも選び直すことになっていた —— そのあいだに 1 つ間違えると、
+    無人で回る層が別の相手に回り続ける。
+
+    **このフォームの submit として出す**(別のフォームにしない)。`<form>` は
+    入れ子にできないので、中にもう 1 枚置くと**内側が丸ごと無視される**
+    (押しても何も起きないボタンになる)。同じフォームなら、**書きかけの欄も
+    一緒に保存してから動く** —— 動かすために保存し直す手間も要らない。
+
+    **端では出さない**(押せないボタンを置かない、の流儀)。空の段(まだ相手を
+    選んでいない末尾の 1 行)にも出さない —— 動かす中身が無い。
+    """
+    if step is None:
+        return ""
+    up = (
+        "" if index == 0 else
+        '<button type="submit" name="step_move" class="step-move"'
+        f' value="up:{index}" title="上へ">↑</button>'
+    )
+    down = (
+        "" if index >= last else
+        '<button type="submit" name="step_move" class="step-move"'
+        f' value="down:{index}" title="下へ">↓</button>'
+    )
+    return up + down
+
+
 def _worker_form(worker: workers.Worker | None, selects, running: str = "") -> str:
     backend_select, model_select = selects
     name = worker.name if worker else ""
     steps = list(worker.steps) if worker else []
     # **空の段を 1 つ足して出す。** 足すのに押す手数を要らなくするため
     rows = [
-        _step_row(i, steps[i] if i < len(steps) else None, backend_select, model_select)
+        _step_row(
+            i, steps[i] if i < len(steps) else None, backend_select, model_select,
+            last=len(steps) - 1,
+        )
         for i in range(min(len(steps) + 1, MAX_STEPS))
     ]
     hint = ("名前を消すと、このワーカーは無くなります" if worker
@@ -247,6 +282,23 @@ async def wake_worker(request: Request):
     return RedirectResponse(BACK_TO_SECTION, status_code=303)
 
 
+def _moved_steps(steps: tuple, raw: str) -> tuple:
+    """`up:1` / `down:0` で段を 1 つ動かした並び。読めない指示は無視する。
+
+    **端は動かさない**(画面は端にボタンを出さないが、押された形は入ってくる)。
+    """
+    kind, _, index = raw.partition(":")
+    if kind not in ("up", "down") or not index.isdigit():
+        return steps
+    at = int(index)
+    to = at - 1 if kind == "up" else at + 1
+    if not (0 <= at < len(steps) and 0 <= to < len(steps)):
+        return steps
+    out = list(steps)
+    out[at], out[to] = out[to], out[at]
+    return tuple(out)
+
+
 @router.post("/admin/ai/workers")
 async def save_worker(request: Request):
     """ワーカー 1 つぶんを保存する。**その 1 つだけ**を書き換える。
@@ -268,6 +320,9 @@ async def save_worker(request: Request):
         workers.Step(str(b).strip(), str(models[i] if i < len(models) else "").strip())
         for i, b in enumerate(backends) if str(b).strip()
     )
+    # **↑↓ はこのフォームの submit。** 書きかけの欄も一緒に保存してから動かす
+    # (`_move_step_html`。`<form>` は入れ子にできないので、別フォームにはできない)
+    steps = _moved_steps(steps, str(form.get("step_move") or ""))
 
     try:
         current = workers.load()

tests/test_workers.py

@@ -1072,3 +1072,74 @@ def test_it_is_not_folded_away(self, enabled):
         html = self._form(workers.Worker("精査", (workers.Step("codex"),)))
 
         assert "<details><summary>待ち行列" not in html
+
+
+class TestReorderingTheSteps:
+    """段の並びは「詰まったら次へ」の順そのもの。**入れ替えるのに打ち直させない。**
+
+    並びを変えるには相手を選び直すしかなく、3 段あれば 3 つとも選び直すことに
+    なっていた —— そのあいだに 1 つ間違えると、無人で回る層が別の相手に回り続ける。
+    """
+
+    def _post(self, **extra):
+        from starlette.datastructures import FormData
+
+        rows = extra.pop("rows", [("codex", ""), ("antigravity", ""), ("claude", "")])
+        items = [("worker_key", "精査"), ("worker_name", "精査")]
+        for backend, model in rows:
+            items += [("step_backend", backend), ("step_model", model)]
+        items += list(extra.items())
+        return FormData(items)
+
+    async def _save(self, form):
+        from app.views import ai_workers
+
+        class _Request:
+            async def form(self):
+                return form
+
+        await ai_workers.save_worker(_Request())
+        return [s.backend for s in workers.load()[0].steps]
+
+    def test_up_swaps_with_the_one_before(self, enabled):
+        got = asyncio.run(self._save(self._post(step_move="up:1")))
+        assert got == ["antigravity", "codex", "claude"]
+
+    def test_down_swaps_with_the_one_after(self, enabled):
+        got = asyncio.run(self._save(self._post(step_move="down:0")))
+        assert got == ["antigravity", "codex", "claude"]
+
+    def test_the_typed_rows_are_kept(self, enabled):
+        """**書きかけの欄も一緒に保存してから動く**(別のフォームにはできない)。"""
+        rows = [("codex", "gpt-6"), ("antigravity", "gemini-3.8-flash")]
+        got = asyncio.run(self._save(self._post(rows=rows, step_move="up:1")))
+        assert got == ["antigravity", "codex"]
+        assert [s.model for s in workers.load()[0].steps] == ["gemini-3.8-flash", "gpt-6"]
+
+    def test_the_ends_do_not_move(self, enabled):
+        """画面は端にボタンを出さないが、押された形は入ってくる。"""
+        assert asyncio.run(self._save(self._post(step_move="up:0"))) == [
+            "codex", "antigravity", "claude"
+        ]
+        assert asyncio.run(self._save(self._post(step_move="down:2"))) == [
+            "codex", "antigravity", "claude"
+        ]
+
+    def test_a_move_we_cannot_read_changes_nothing(self, enabled):
+        for raw in ("", "sideways:1", "up:", "up:x", "up:99"):
+            assert asyncio.run(self._save(self._post(step_move=raw))) == [
+                "codex", "antigravity", "claude"
+            ], raw
+
+    def test_the_screen_shows_the_arrows_only_where_they_work(self, enabled):
+        from app.views import admin, ai_workers
+
+        workers.save([_worker(workers.Step("codex"), workers.Step("antigravity"))])
+        html = ai_workers.section_html(
+            (admin._backend_select, admin._model_select)
+        )
+
+        # 2 段あるので ↑ は 2 番目だけ、↓ は 1 番目だけ。空の 3 行目には出ない
+        assert 'value="up:1"' in html and 'value="up:0"' not in html
+        assert 'value="down:0"' in html and 'value="down:1"' not in html
+        assert 'value="down:2"' not in html and 'value="up:2"' not in html