diff --git a/lib/bds/ai.ex b/lib/bds/ai.ex
index be94f9d..b41c997 100644
--- a/lib/bds/ai.ex
+++ b/lib/bds/ai.ex
@@ -317,6 +317,48 @@ defmodule BDS.AI do
|> Enum.map(&format_conversation/1)
end
+ def available_chat_models(current_model \\ nil) do
+ endpoint_models =
+ [:online, :airplane]
+ |> Enum.flat_map(fn kind ->
+ case get_endpoint(kind) do
+ {:ok, %{model: model}} when is_binary(model) and model != "" -> [model]
+ _other -> []
+ end
+ end)
+
+ preference_models =
+ [:chat, :airplane_chat]
+ |> Enum.flat_map(fn key ->
+ case get_model_preference(key) do
+ {:ok, model} when is_binary(model) and model != "" -> [model]
+ _other -> []
+ end
+ end)
+
+ [current_model | endpoint_models ++ preference_models]
+ |> Enum.filter(&(is_binary(&1) and String.trim(&1) != ""))
+ |> Enum.uniq()
+ |> Enum.map(&%{id: &1, name: &1})
+ end
+
+ def set_conversation_model(conversation_id, model_id)
+ when is_binary(conversation_id) and is_binary(model_id) do
+ case Repo.get(ChatConversation, conversation_id) do
+ nil ->
+ {:error, :not_found}
+
+ %ChatConversation{} = conversation ->
+ conversation
+ |> ChatConversation.changeset(%{model: model_id, updated_at: Persistence.now_ms()})
+ |> Repo.update()
+ |> case do
+ {:ok, updated_conversation} -> {:ok, format_conversation(updated_conversation)}
+ error -> error
+ end
+ end
+ end
+
def list_chat_messages(conversation_id) when is_binary(conversation_id) do
Repo.all(
from message in ChatMessage,
diff --git a/lib/bds/desktop/shell_commands.ex b/lib/bds/desktop/shell_commands.ex
index 4347290..b616796 100644
--- a/lib/bds/desktop/shell_commands.ex
+++ b/lib/bds/desktop/shell_commands.ex
@@ -514,14 +514,10 @@ defmodule BDS.Desktop.ShellCommands do
defp normalize_translation_validation(report) do
%{
- summary: %{
- missing_count: length(report.missing),
- orphan_count: length(report.orphan_files),
- do_not_translate_count: length(report.do_not_translate_posts)
- },
- missing: Enum.map(report.missing, &stringify_map/1),
- orphan_files: report.orphan_files,
- do_not_translate_posts: report.do_not_translate_posts
+ checked_database_row_count: report.checked_database_row_count,
+ checked_filesystem_file_count: report.checked_filesystem_file_count,
+ invalid_database_rows: Enum.map(report.invalid_database_rows, &stringify_map/1),
+ invalid_filesystem_files: Enum.map(report.invalid_filesystem_files, &stringify_map/1)
}
end
@@ -532,11 +528,10 @@ defmodule BDS.Desktop.ShellCommands do
project_id: project_id,
route: "translation_validation",
title: "Translation Validation",
- subtitle: "Published posts checked against required blog languages",
+ subtitle: "Database rows and translation files checked for invalid state",
editorMeta: [
- %{label: "Missing", value: Integer.to_string(length(report.missing))},
- %{label: "Orphan Files", value: Integer.to_string(length(report.orphan_files))},
- %{label: "Skipped", value: Integer.to_string(length(report.do_not_translate_posts))}
+ %{label: "Invalid DB", value: Integer.to_string(length(report.invalid_database_rows))},
+ %{label: "Invalid Files", value: Integer.to_string(length(report.invalid_filesystem_files))}
],
payload: normalize_translation_validation(report)
}
diff --git a/lib/bds/desktop/shell_live.ex b/lib/bds/desktop/shell_live.ex
index c517c36..2e4f23e 100644
--- a/lib/bds/desktop/shell_live.ex
+++ b/lib/bds/desktop/shell_live.ex
@@ -97,7 +97,9 @@ defmodule BDS.Desktop.ShellLive do
|> assign(:script_editor_drafts, %{})
|> assign(:template_editor_drafts, %{})
|> assign(:chat_editor_inputs, %{})
+ |> assign(:chat_model_selectors_open, %{})
|> assign(:misc_editor_selected_pairs, %{})
+ |> assign(:misc_editor_git_selected_files, %{})
|> assign(:metadata_diff_active_tabs, %{})
|> assign(:metadata_diff_field_filters, %{})
|> assign(:shell_overlay, nil)
@@ -683,6 +685,14 @@ defmodule BDS.Desktop.ShellLive do
{:noreply, ChatEditor.update_input(socket, message, &reload_shell/2)}
end
+ def handle_event("toggle_chat_model_selector", _params, socket) do
+ {:noreply, ChatEditor.toggle_model_selector(socket, &reload_shell/2)}
+ end
+
+ def handle_event("select_chat_model", %{"model" => model_id}, socket) do
+ {:noreply, ChatEditor.set_model(socket, model_id, &reload_shell/2, &append_output_entry/5)}
+ end
+
def handle_event("send_chat_editor_message", _params, socket) do
{:noreply, ChatEditor.send_message(socket, &reload_shell/2, &append_output_entry/5)}
end
@@ -701,6 +711,17 @@ defmodule BDS.Desktop.ShellLive do
end
end
+ def handle_event("fix_translation_validation", _params, socket) do
+ case MiscEditor.fix_translation_validation(socket, &append_output_entry/5) do
+ {:rerun, next_socket} -> {:noreply, apply_shell_command(next_socket, "validate_translations")}
+ {:socket, next_socket} -> {:noreply, next_socket}
+ end
+ end
+
+ def handle_event("select_git_diff_file", %{"path" => path}, socket) do
+ {:noreply, socket |> MiscEditor.select_git_diff_file(path) |> assign_misc_editor()}
+ end
+
def handle_event("toggle_duplicate_pair", %{"pair-id" => pair_id}, socket) do
{:noreply, MiscEditor.toggle_duplicate(socket, pair_id, &reload_shell/2)}
end
diff --git a/lib/bds/desktop/shell_live/chat_editor.ex b/lib/bds/desktop/shell_live/chat_editor.ex
index 04f5a59..6d92b2b 100644
--- a/lib/bds/desktop/shell_live/chat_editor.ex
+++ b/lib/bds/desktop/shell_live/chat_editor.ex
@@ -13,6 +13,31 @@ defmodule BDS.Desktop.ShellLive.ChatEditor do
assign(socket, :chat_editor, build(socket.assigns))
end
+ def toggle_model_selector(socket, reload) do
+ %{id: conversation_id} = socket.assigns.current_tab
+ current = Map.get(socket.assigns.chat_model_selectors_open, conversation_id, false)
+
+ socket
+ |> assign(:chat_model_selectors_open, Map.put(socket.assigns.chat_model_selectors_open, conversation_id, not current))
+ |> reload.(socket.assigns.workbench)
+ end
+
+ def set_model(socket, model_id, reload, append_output) do
+ %{id: conversation_id} = socket.assigns.current_tab
+
+ case AI.set_conversation_model(conversation_id, model_id) do
+ {:ok, _conversation} ->
+ socket
+ |> assign(:chat_model_selectors_open, Map.put(socket.assigns.chat_model_selectors_open, conversation_id, false))
+ |> reload.(socket.assigns.workbench)
+
+ {:error, reason} ->
+ socket
+ |> append_output.(translated("Chat"), inspect(reason), nil, "error")
+ |> reload.(socket.assigns.workbench)
+ end
+ end
+
def update_input(socket, value, reload) do
%{id: conversation_id} = socket.assigns.current_tab
@@ -53,10 +78,12 @@ defmodule BDS.Desktop.ShellLive.ChatEditor do
%ChatConversation{} = conversation ->
%{
id: conversation.id,
- title: conversation.title || translated("New Chat"),
+ title: conversation.title || translated("chat.newChat"),
model: conversation.model,
+ available_models: AI.available_chat_models(conversation.model),
+ model_selector_open?: Map.get(assigns.chat_model_selectors_open, conversation.id, false),
input: Map.get(assigns.chat_editor_inputs, conversation.id, ""),
- messages: AI.list_chat_messages(conversation.id),
+ messages: build_entries(AI.list_chat_messages(conversation.id)),
offline?: Map.get(assigns, :offline_mode, true)
}
end
@@ -64,5 +91,89 @@ defmodule BDS.Desktop.ShellLive.ChatEditor do
def build(_assigns), do: nil
+ def message_role_label(:user), do: translated("chat.role.you")
+ def message_role_label(_role), do: translated("chat.role.assistant")
+
+ def tool_call_name(tool_call) when is_map(tool_call) do
+ Map.get(tool_call, "name") || Map.get(tool_call, :name) || "tool"
+ end
+
+ def tool_surface_type(surface), do: Map.get(surface, :type, "json")
+
+ defp build_entries(messages) do
+ {entries, current_entry} =
+ Enum.reduce(messages, {[], nil}, fn message, {entries, current_entry} ->
+ case message.role do
+ :tool ->
+ if current_entry && current_entry.role == :assistant do
+ {entries, append_tool_surface(current_entry, message)}
+ else
+ {entries, current_entry}
+ end
+
+ :system ->
+ {entries, current_entry}
+
+ _other ->
+ entries = finalize_entry(entries, current_entry)
+ {entries, start_entry(message)}
+ end
+ end)
+
+ entries
+ |> finalize_entry(current_entry)
+ |> Enum.reverse()
+ end
+
+ defp finalize_entry(entries, nil), do: entries
+ defp finalize_entry(entries, entry), do: [entry | entries]
+
+ defp start_entry(message) do
+ %{
+ id: message.id,
+ role: message.role,
+ content: message.content || "",
+ tool_markers: normalize_tool_calls(message.tool_calls),
+ tool_surfaces: []
+ }
+ end
+
+ defp append_tool_surface(entry, message) do
+ case normalize_tool_surface(message.content) do
+ nil -> entry
+ surface -> update_in(entry.tool_surfaces, &(&1 ++ [surface]))
+ end
+ end
+
+ defp normalize_tool_calls(tool_calls) when is_list(tool_calls) do
+ Enum.map(tool_calls, fn tool_call ->
+ %{
+ name: tool_call_name(tool_call),
+ arguments: Map.get(tool_call, "arguments") || Map.get(tool_call, :arguments) || Map.get(tool_call, "args") || Map.get(tool_call, :args) || %{}
+ }
+ end)
+ end
+
+ defp normalize_tool_calls(_tool_calls), do: []
+
+ defp normalize_tool_surface(content) when is_binary(content) do
+ case Jason.decode(content) do
+ {:ok, %{"type" => type} = decoded} ->
+ %{
+ type: type,
+ title: decoded["title"],
+ columns: List.wrap(decoded["columns"]),
+ rows: Enum.map(List.wrap(decoded["rows"]), &List.wrap/1),
+ fields: List.wrap(decoded["fields"]),
+ data: decoded
+ }
+
+ _other ->
+ nil
+ end
+ end
+
+ defp normalize_tool_surface(_content), do: nil
+
def translated(text, bindings \\ %{}), do: ShellData.translate(text, bindings, Process.get(:bds_ui_locale))
end
diff --git a/lib/bds/desktop/shell_live/chat_editor_html/chat_editor.html.heex b/lib/bds/desktop/shell_live/chat_editor_html/chat_editor.html.heex
index a87db3e..35674b9 100644
--- a/lib/bds/desktop/shell_live/chat_editor_html/chat_editor.html.heex
+++ b/lib/bds/desktop/shell_live/chat_editor_html/chat_editor.html.heex
@@ -1,37 +1,113 @@
<%= if Enum.empty?(@chat_editor.messages) do %>
🤖
-
<%= translated("New Chat") %>
-
<%= translated("Ask the assistant about the active project.") %>
+
<%= translated("chat.welcomeTitle") %>
+
<%= translated("chat.welcomeDescription") %>
- - <%= translated("Search posts and media") %>
- - <%= translated("Inspect metadata") %>
- - <%= translated("Open related tabs") %>
- - <%= translated("Review generated output") %>
- - <%= translated("Navigate settings") %>
+ - <%= translated("chat.welcomeTipSearch") %>
+ - <%= translated("chat.welcomeTipChart") %>
+ - <%= translated("chat.welcomeTipTable") %>
+ - <%= translated("chat.welcomeTipMetadata") %>
+ - <%= translated("chat.welcomeTipTabs") %>
<% else %>
<%= for message <- @chat_editor.messages do %>
-
+
+
+ <%= if message.tool_markers != [] do %>
+
+ <% end %>
+
<%= message.content || "" %>
+
+ <%= for surface <- message.tool_surfaces do %>
+
+ <%= if surface.title do %>
+ <%= surface.title %>
+ <% end %>
+
+ <%= case tool_surface_type(surface) do %>
+ <% "table" -> %>
+
+
+ <% _other -> %>
+ <%= Jason.encode!(surface.data, pretty: true) %>
+ <% end %>
+
+ <% end %>
<% end %>
<% end %>
diff --git a/lib/bds/desktop/shell_live/misc_editor.ex b/lib/bds/desktop/shell_live/misc_editor.ex
index d9ae3cf..26d86fb 100644
--- a/lib/bds/desktop/shell_live/misc_editor.ex
+++ b/lib/bds/desktop/shell_live/misc_editor.ex
@@ -3,8 +3,11 @@ defmodule BDS.Desktop.ShellLive.MiscEditor do
use Phoenix.Component
- alias BDS.{Embeddings, Generation, Git}
+ import Ecto.Query
+
+ alias BDS.{Embeddings, Generation, Git, Posts, Repo}
alias BDS.Desktop.ShellData
+ alias BDS.Settings.Setting
embed_templates "misc_editor_html/*"
@@ -114,6 +117,37 @@ defmodule BDS.Desktop.ShellLive.MiscEditor do
end
end
+ def fix_translation_validation(socket, append_output) do
+ report =
+ socket.assigns
+ |> meta()
+ |> Map.get(:payload, %{})
+ |> normalize_translation_validation_report()
+
+ {:ok, result} = Posts.fix_invalid_translations(report)
+
+ {:rerun,
+ socket
+ |> append_output.(
+ translated("Translation Validation"),
+ translated("translationValidation.toast.fixSuccess", %{
+ dbRows: result.deleted_database_rows,
+ files: result.deleted_files,
+ flushed: result.flushed_translations
+ })
+ )}
+ rescue
+ error -> {:socket, append_output.(socket, translated("Translation Validation"), inspect(error), nil, "error")}
+ end
+
+ def select_git_diff_file(socket, file_path) do
+ assign(
+ socket,
+ :misc_editor_git_selected_files,
+ Map.put(socket.assigns.misc_editor_git_selected_files, socket.assigns.current_tab.id, file_path)
+ )
+ end
+
def metadata_diff_repair_request(socket, field, direction) do
meta = meta(socket.assigns)
payload = Map.get(meta, :payload, %{})
@@ -245,14 +279,23 @@ defmodule BDS.Desktop.ShellLive.MiscEditor do
end
defp build_translation_validation(meta, payload) do
+ report = normalize_translation_validation_report(payload)
+
%{
kind: :translation_validation,
title: Map.get(meta, :title, translated("Translation Validation")),
subtitle: Map.get(meta, :subtitle, ""),
- summary: Map.get(payload, :summary, %{}),
- missing: Map.get(payload, :missing, []),
- orphan_files: Map.get(payload, :orphan_files, []),
- do_not_translate_posts: Map.get(payload, :do_not_translate_posts, [])
+ summary: %{},
+ summary_text:
+ translated("translationValidation.summary", %{
+ dbRows: report.checked_database_row_count,
+ files: report.checked_filesystem_file_count,
+ invalidDb: length(report.invalid_database_rows),
+ invalidFiles: length(report.invalid_filesystem_files)
+ }),
+ invalid_database_rows: report.invalid_database_rows,
+ invalid_filesystem_files: report.invalid_filesystem_files,
+ can_fix?: report.invalid_database_rows != [] or report.invalid_filesystem_files != []
}
end
@@ -270,29 +313,93 @@ defmodule BDS.Desktop.ShellLive.MiscEditor do
end
defp build_git_diff(assigns, meta) do
- diff_text =
- case Git.diff(assigns.projects.active_project_id) do
- {:ok, %{staged_diff: staged, unstaged_diff: unstaged}} ->
- [
- "# Staged Changes\n\n",
- if(String.trim(staged) == "", do: translated("No staged changes"), else: staged),
- "\n\n# Working Tree\n\n",
- if(String.trim(unstaged) == "", do: translated("No unstaged changes"), else: unstaged)
- ]
- |> IO.iodata_to_binary()
+ project_id = assigns.projects.active_project_id
+ selected_files = Map.get(assigns, :misc_editor_git_selected_files, %{})
- {:error, reason} -> inspect(reason)
+ {files, diff, error_message} =
+ case Git.status(project_id) do
+ {:ok, %{files: files}} ->
+ file_paths = files |> Enum.map(&Map.get(&1, :path)) |> Enum.reject(&is_nil/1) |> Enum.uniq() |> Enum.sort()
+ selected_file_path = select_git_diff_path(assigns.current_tab.id, file_paths, selected_files)
+
+ diff =
+ case selected_file_path do
+ nil ->
+ empty_git_diff(project_id)
+
+ file_path ->
+ case Git.get_diff_content(project_id, file_path) do
+ {:ok, diff} -> diff
+ {:error, reason} -> Map.merge(empty_git_diff(project_id), %{file_path: file_path, error: inspect(reason)})
+ end
+ end
+
+ {file_paths, diff, nil}
+
+ {:error, reason} ->
+ {[], empty_git_diff(project_id), inspect(reason)}
end
+ preferences = git_diff_preferences()
+
%{
kind: :git_diff,
title: Map.get(meta, :title, translated("Git Diff")),
subtitle: Map.get(meta, :subtitle, ""),
- diff_text: diff_text,
- summary: %{}
+ summary: %{},
+ files: files,
+ selected_file_path: diff.file_path,
+ active_diff: Map.put(diff, :language, git_diff_language(diff.file_path)),
+ preferences: preferences,
+ empty_message: error_message || translated("No unstaged changes")
}
end
+ def translation_issue_label(issue) do
+ case issue_value(issue, :issue) do
+ "same-language-as-canonical" -> translated("translationValidation.issue.sameLanguage")
+ "do-not-translate-has-translations" -> translated("translationValidation.issue.doNotTranslate")
+ "content-in-database" -> translated("translationValidation.issue.contentInDatabase")
+ _other -> translated("translationValidation.issue.missingSource")
+ end
+ end
+
+ def translation_issue_languages(issue) do
+ canonical_language = issue_value(issue, :canonical_language)
+ translation_language = issue_value(issue, :translation_language)
+
+ if canonical_language in [nil, ""] do
+ translation_language
+ else
+ translated("translationValidation.languagesWithCanonical", %{
+ canonical: canonical_language,
+ translation: translation_language
+ })
+ end
+ end
+
+ def translation_issue_value(issue, key), do: issue_value(issue, key)
+
+ def git_diff_language(nil), do: "plaintext"
+
+ def git_diff_language(file_path) do
+ case file_path |> Path.extname() |> String.downcase() do
+ ".md" -> "markdown"
+ ".markdown" -> "markdown"
+ ".mdx" -> "markdown"
+ ".ts" -> "typescript"
+ ".tsx" -> "typescript"
+ ".js" -> "javascript"
+ ".jsx" -> "javascript"
+ ".json" -> "json"
+ ".css" -> "css"
+ ".html" -> "html"
+ ".yml" -> "yaml"
+ ".yaml" -> "yaml"
+ _other -> "plaintext"
+ end
+ end
+
defp meta(assigns) do
Map.get(assigns.tab_meta, {assigns.current_tab.type, assigns.current_tab.id}, %{})
end
@@ -496,4 +603,60 @@ defmodule BDS.Desktop.ShellLive.MiscEditor do
defp diff_name(diff) do
Map.get(diff, :field) || Map.get(diff, "field") || Map.get(diff, :name) || Map.get(diff, "name") || "value"
end
+
+ defp normalize_translation_validation_report(payload) when is_map(payload) do
+ %{
+ checked_database_row_count: issue_value(payload, :checked_database_row_count) || 0,
+ checked_filesystem_file_count: issue_value(payload, :checked_filesystem_file_count) || 0,
+ invalid_database_rows:
+ payload
+ |> issue_value(:invalid_database_rows)
+ |> List.wrap(),
+ invalid_filesystem_files:
+ payload
+ |> issue_value(:invalid_filesystem_files)
+ |> List.wrap()
+ }
+ end
+
+ defp normalize_translation_validation_report(_payload) do
+ %{
+ checked_database_row_count: 0,
+ checked_filesystem_file_count: 0,
+ invalid_database_rows: [],
+ invalid_filesystem_files: []
+ }
+ end
+
+ defp issue_value(issue, key) when is_map(issue) do
+ Map.get(issue, key) || Map.get(issue, Atom.to_string(key))
+ end
+
+ defp issue_value(_issue, _key), do: nil
+
+ defp select_git_diff_path(tab_id, files, selected_files) do
+ selected = Map.get(selected_files, tab_id)
+
+ if selected in files do
+ selected
+ else
+ List.first(files)
+ end
+ end
+
+ defp empty_git_diff(file_path) do
+ %{file_path: file_path, original: "", modified: "", error: nil}
+ end
+
+ defp git_diff_preferences do
+ %{
+ view_style: get_global_setting("ui.git_diff_view_style") || "inline",
+ word_wrap: get_global_setting("ui.git_diff_word_wrap") == "true",
+ hide_unchanged_regions: get_global_setting("ui.git_diff_hide_unchanged_regions") == "true"
+ }
+ end
+
+ defp get_global_setting(key) do
+ Repo.one(from setting in Setting, where: setting.key == ^key, select: setting.value)
+ end
end
diff --git a/lib/bds/desktop/shell_live/misc_editor_html/misc_editor.html.heex b/lib/bds/desktop/shell_live/misc_editor_html/misc_editor.html.heex
index eb2ad06..87b8a1e 100644
--- a/lib/bds/desktop/shell_live/misc_editor_html/misc_editor.html.heex
+++ b/lib/bds/desktop/shell_live/misc_editor_html/misc_editor.html.heex
@@ -181,10 +181,79 @@
<% :translation_validation -> %>
-
-
<%= translated("Missing") %>
<%= for issue <- @misc_editor.missing do %>- <%= inspect(issue) %>
<% end %>
-
<%= translated("Orphan Files") %>
<%= for file <- @misc_editor.orphan_files do %>- <%= file %>
<% end %>
-
<%= translated("Do Not Translate") %>
<%= for post <- @misc_editor.do_not_translate_posts do %>- <%= inspect(post) %>
<% end %>
+
+
+ <%= @misc_editor.summary_text %>
+
+
+
+ <%= translated("translationValidation.databaseTitle") %>
+
+ <%= if @misc_editor.invalid_database_rows == [] do %>
+ <%= translated("translationValidation.noneDatabase") %>
+ <% else %>
+
+ <%= for issue <- @misc_editor.invalid_database_rows do %>
+
+ <%= translation_issue_label(issue) %>
+
+ - <%= translated("translationValidation.field.translationFor") %>
+ - <%= translation_issue_value(issue, :translation_for) %>
+ <%= if translation_issue_value(issue, :translation_id) do %>
+ - <%= translated("translationValidation.field.translationId") %>
+ - <%= translation_issue_value(issue, :translation_id) %>
+ <% end %>
+ <%= if translation_issue_value(issue, :title) do %>
+ - <%= translated("translationValidation.field.title") %>
+ - <%= translation_issue_value(issue, :title) %>
+ <% end %>
+ - <%= translated("translationValidation.field.languages") %>
+ - <%= translation_issue_languages(issue) %>
+ <%= if translation_issue_value(issue, :file_path) do %>
+ - <%= translated("translationValidation.field.filePath") %>
+ - <%= translation_issue_value(issue, :file_path) %>
+ <% end %>
+
+
+ <% end %>
+
+
+
+ <%= translated("translationValidation.filesystemTitle") %>
+
+ <%= if @misc_editor.invalid_filesystem_files == [] do %>
+ <%= translated("translationValidation.noneFilesystem") %>
+ <% else %>
+
+ <%= for issue <- @misc_editor.invalid_filesystem_files do %>
+
+ <%= translation_issue_label(issue) %>
+
+ - <%= translated("translationValidation.field.translationFor") %>
+ - <%= translation_issue_value(issue, :translation_for) %>
+ <%= if translation_issue_value(issue, :title) do %>
+ - <%= translated("translationValidation.field.title") %>
+ - <%= translation_issue_value(issue, :title) %>
+ <% end %>
+ - <%= translated("translationValidation.field.languages") %>
+ - <%= translation_issue_languages(issue) %>
+ <%= if translation_issue_value(issue, :file_path) do %>
+ - <%= translated("translationValidation.field.filePath") %>
+ - <%= translation_issue_value(issue, :file_path) %>
+ <% end %>
+
+
+ <% end %>
+
+
+
+
+
+
<% :find_duplicates -> %>
@@ -202,7 +271,35 @@
<%= @misc_editor.diff_text %>
+ <%= if @misc_editor.files == [] do %>
+
<%= @misc_editor.empty_message %>
+ <% else %>
+
+
+
+ <% end %>
+
)
+ end
+
test "settings sidebar categories render the full old-app section model and target the requested section" do
{:ok, view, _html} = live_isolated(build_conn(), BDS.Desktop.ShellLive)
diff --git a/test/bds/git_test.exs b/test/bds/git_test.exs
index e2e1cb7..b9552b2 100644
--- a/test/bds/git_test.exs
+++ b/test/bds/git_test.exs
@@ -72,6 +72,26 @@ defmodule BDS.GitTest do
assert repo.current_branch == "main"
end
+ test "get_diff_content returns HEAD and working tree content for a changed file", %{
+ project: project,
+ project_dir: project_dir
+ } do
+ posts_dir = Path.join(project_dir, "posts")
+ File.mkdir_p!(posts_dir)
+
+ relative_path = "posts/first.md"
+ full_path = Path.join(project_dir, relative_path)
+
+ File.write!(full_path, "Old content\n")
+ init_git_repo!(project_dir, "initial")
+ File.write!(full_path, "New content\n")
+
+ assert {:ok, diff} = Git.get_diff_content(project.id, relative_path)
+ assert diff.file_path == relative_path
+ assert diff.original == "Old content\n"
+ assert diff.modified == "New content\n"
+ end
+
test "remote_state reports upstream ahead and behind counts", %{project: project} do
runner = fake_runner(fn
"git", ["rev-parse", "--abbrev-ref", "HEAD"], _opts -> {"main\n", 0}
@@ -146,4 +166,18 @@ defmodule BDS.GitTest do
defp fake_runner(handler) do
fn command, args, opts -> handler.(command, args, opts) end
end
+
+ defp init_git_repo!(project_dir, message) do
+ run_git!(project_dir, ["init", "-b", "master"])
+ run_git!(project_dir, ["config", "user.name", "bDS Tests"])
+ run_git!(project_dir, ["config", "user.email", "tests@example.com"])
+ run_git!(project_dir, ["add", "-A"])
+ run_git!(project_dir, ["commit", "-m", message])
+ end
+
+ defp run_git!(dir, args) do
+ {output, status} = System.cmd("git", args, cd: dir, stderr_to_stdout: true)
+
+ assert status == 0, output
+ end
end
diff --git a/test/bds/posts_test.exs b/test/bds/posts_test.exs
index 2e868b4..c3dcfee 100644
--- a/test/bds/posts_test.exs
+++ b/test/bds/posts_test.exs
@@ -705,6 +705,75 @@ defmodule BDS.PostsTest do
assert {:ok, %{indexed: 3, total: 3}} = BDS.Embeddings.get_indexing_progress(project.id)
end
+ test "validate_translations and fix_invalid_translations follow the legacy invalid-translation workflow",
+ %{project: project} do
+ assert {:ok, post} =
+ BDS.Posts.create_post(%{
+ project_id: project.id,
+ title: "Source Post",
+ content: "Canonical body",
+ language: "en"
+ })
+
+ assert {:ok, published_post} = BDS.Posts.publish_post(post.id)
+
+ assert {:ok, translation} =
+ BDS.Posts.upsert_post_translation(post.id, "de", %{
+ title: "Translated Post",
+ content: "Translated body",
+ status: :published
+ })
+
+ invalid_file_path =
+ Path.join([
+ BDS.Projects.project_data_dir(project),
+ Path.dirname(published_post.file_path),
+ "#{published_post.slug}.en.md"
+ ])
+
+ File.write!(
+ invalid_file_path,
+ [
+ "---",
+ "translationFor: #{post.id}",
+ "language: en",
+ "title: Invalid Same Language",
+ "---",
+ "Wrong translation",
+ ""
+ ]
+ |> Enum.join("\n")
+ )
+
+ assert {:ok, report} = BDS.Posts.validate_translations(project.id)
+
+ assert report.checked_database_row_count == 1
+ assert report.checked_filesystem_file_count == 1
+
+ assert [db_issue] = report.invalid_database_rows
+ assert db_issue.issue == "content-in-database"
+ assert db_issue.translation_id == translation.id
+ assert db_issue.translation_for == post.id
+ assert db_issue.translation_language == "de"
+
+ assert [file_issue] = report.invalid_filesystem_files
+ assert file_issue.issue == "same-language-as-canonical"
+ assert file_issue.translation_for == post.id
+ assert file_issue.translation_language == "en"
+ assert file_issue.file_path == invalid_file_path
+
+ assert {:ok, result} = BDS.Posts.fix_invalid_translations(report)
+ assert result.deleted_database_rows == 0
+ assert result.deleted_files == 1
+ assert result.flushed_translations == 1
+
+ saved_translation = BDS.Repo.get!(BDS.Posts.Translation, translation.id)
+ assert saved_translation.content == nil
+ assert is_binary(saved_translation.file_path)
+ assert File.exists?(Path.join(BDS.Projects.project_data_dir(project), saved_translation.file_path))
+ refute File.exists?(invalid_file_path)
+ end
+
def handle_repo_query(_event, _measurements, metadata, owner_pid) do
send(owner_pid, {:repo_query, metadata.query || ""})
end