Renaming an assistant renames it in its reviews and events; settings pages return to the section saved ed89fae5180b

  /* Settings pages: a form posts and the page comes back from the top.

     Remember which section's form was sent, and on the same page's next

     load scroll back to it, bringing the notice along. */

  document.addEventListener("submit", function (e) {

    var sec = e.target.closest ? e.target.closest("section[id], form[id], details[id], div[id]") : null;

    if (!sec || !sec.id) return;

    try { sessionStorage.setItem("oxy-return", location.pathname + "#" + sec.id); } catch (err) { /* private mode */ }

  });

  try {

    var back = sessionStorage.getItem("oxy-return");

    if (back && back.indexOf(location.pathname + "#") === 0) {

      sessionStorage.removeItem("oxy-return");

      var target = document.getElementById(back.split("#")[1]);

      if (target) {

        var flash = document.querySelector("main > .note, main > .error, .sections > .note, .sections > .error");

        if (flash && !target.contains(flash)) target.parentNode.insertBefore(flash, target);

        target.scrollIntoView({ block: "start" });

      }

    }

  } catch (err) { /* no storage: nothing to restore */ }

/// A renamed assistant keeps its name on the reviews it answered and the

/// events that mention it, for the repositories its level covers: the

/// name is what people read, not a key.

pub async fn rename_in_history(pool: &Pool, scope: Scope, owner_hint: &str, old: &str, new: &str) -> sqlx::Result<()> {

    if old == new || old.is_empty() || new.is_empty() {

        return Ok(());

    }

    let like = format!("%{old}%");

    match scope {

        Scope::Site => {

            sqlx::query("UPDATE proposal_reviews SET via = ? WHERE via = ?").bind(new).bind(old).execute(pool).await?;

            sqlx::query("UPDATE proposal_events SET event = replace(event, ?, ?) WHERE event LIKE ?")

                .bind(old)

                .bind(new)

                .bind(&like)

                .execute(pool)

                .await?;

        }

        Scope::Org(_) => {

            let in_org = "proposal_id IN (SELECT p.id FROM proposals p JOIN repositories r ON r.id = p.repo_id WHERE r.owner = ?)";

            sqlx::query(&format!("UPDATE proposal_reviews SET via = ? WHERE via = ? AND {in_org}"))

                .bind(new)

                .bind(old)

                .bind(owner_hint)

                .execute(pool)

                .await?;

            sqlx::query(&format!("UPDATE proposal_events SET event = replace(event, ?, ?) WHERE event LIKE ? AND {in_org}"))

                .bind(old)

                .bind(new)

                .bind(&like)

                .bind(owner_hint)

                .execute(pool)

                .await?;

        }

        Scope::Repo(id) => {

            let in_repo = "proposal_id IN (SELECT id FROM proposals WHERE repo_id = ?)";

            sqlx::query(&format!("UPDATE proposal_reviews SET via = ? WHERE via = ? AND {in_repo}"))

                .bind(new)

                .bind(old)

                .bind(id)

                .execute(pool)

                .await?;

            sqlx::query(&format!("UPDATE proposal_events SET event = replace(event, ?, ?) WHERE event LIKE ? AND {in_repo}"))

                .bind(old)

                .bind(new)

                .bind(&like)

                .bind(id)

                .execute(pool)

                .await?;

        }

    }

    Ok(())

}

            if a.name != fields.name {

                assistants::rename_in_history(&app.db, at.scope, &at.owner, &a.name, &fields.name).await?;

            }

    assert!(!page.contains("Ask Claude"), "{page}");

    assert!(page.contains("Draft with Claude"), "{page}");

    assert!(page.contains("Draft with Claude"), "{page}");

    assert!(page.contains("Ask Claude"), "{page}");

    // Renaming the assistant renames it on the review and in the events.

    let admin = get(app, "/admin", Some(&andy)).await.body;

    let r = post(

        app,

        "/admin/assistants",

        Some(&andy),

        &format!(

            "_csrf={}&action=update&id={}&name=Claude+Code&endpoint={ep}&api=chat-completions&model=claude-opus-5-5&token=&enabled=1",

            csrf_of(&admin),

            oxygenhub::assistants::list(&app.db, oxygenhub::levels::Scope::Site).await.unwrap()[0].id

        ),

    )

    .await;

    assert!(r.body.contains("Claude Code updated."), "{}", r.body);

    let page = get(app, "/andy/demo/proposal/1", Some(&andy)).await.body;

    assert!(page.contains("<strong>Claude Code</strong>") && page.contains("asked Claude Code to review"), "{page}");

    assert!(!page.contains("<strong>Claude</strong>"), "{page}");