{# FE2-3: /corpus/{doi} — per-paper detail page.

   Context:
     paper      — dict from _get_paper_row(): doi, title, authors (list), year,
                  journal, source_type, zotero_key, note_path, extraction (dict).
     breakdown  — dict from _get_score_breakdown() ({doi, run_id, breakdown:{...}})
                  or None.

   All paper / extraction content is rendered through Jinja autoescape ONLY —
   never |safe. The score_decomposition.html partial expects a different shape
   (paper.score_breakdown_json string + paper.score), so it is NOT reused here;
   a small table is rendered from breakdown.breakdown instead.
#}
{% extends "base.html" %}
{% block title %}{{ paper.title or "(untitled)" }} — Corpus — lit-monitor{% endblock %}
{% block content %}
<section class="dashboard paper-detail">
  <div class="page-header">
    <p class="field-note"><a href="/corpus">&larr; Back to corpus</a></p>
    <h1>{{ paper.title or "(untitled)" }}</h1>
    <p class="paper-meta field-note">
      {% if paper.authors %}{{ paper.authors | join(", ") }}{% endif %}
      {% if paper.year %} &middot; {{ paper.year }}{% endif %}
      {% if paper.journal %} &middot; {{ paper.journal }}{% endif %}
      {% if paper.source_type %} &middot; <span class="badge">{{ paper.source_type }}</span>{% endif %}
    </p>
    <p class="paper-meta field-note">
      {% if paper.doi %}
        <a class="doi" href="https://doi.org/{{ paper.doi }}" target="_blank" rel="noopener">{{ paper.doi }}</a>
      {% else %}
        <span class="doi">(no DOI)</span>
      {% endif %}
    </p>

    <p class="button-row paper-links">
      {% if paper.zotero_key %}
      <a class="btn btn-secondary btn-sm"
         href="zotero://select/library/items/{{ paper.zotero_key }}">Open in Zotero</a>
      {% endif %}
      {% if paper.note_path %}
      {# PF-2: when a vault is configured, obsidian_uri is an
         obsidian://open?vault=…&file=… deep link (built server-side in
         _obsidian_uri); the readable note path is the link label. When no vault
         is configured obsidian_uri is None and we fall back to the plain path.
         The obsidian:// href passes through Jinja autoescape safely — no |safe. #}
      {% if obsidian_uri %}
      <span class="field-note note-path">Obsidian note:
        <a href="{{ obsidian_uri }}"><code>{{ paper.note_path }}</code></a></span>
      {% else %}
      <span class="field-note note-path">Obsidian note: <code>{{ paper.note_path }}</code></span>
      {% endif %}
      {% endif %}
    </p>
  </div>

  {# --- Extraction -------------------------------------------------------- #}
  <div class="card">
    <h2>Extraction</h2>
    {% if paper.extraction and paper.extraction.get("_overall_confidence") is not none %}
    <p class="field-note">Overall confidence:
      {{ "%.3f" | format(paper.extraction.get("_overall_confidence") | float) }}</p>
    {% endif %}
    {# Collect non-underscore keys; "_"-prefixed keys (e.g. _overall_confidence)
       are metadata, surfaced separately above and skipped here. #}
    {% set _visible = [] %}
    {% for key, value in paper.extraction.items() %}
      {% if not key.startswith("_") %}{% set _ = _visible.append((key, value)) %}{% endif %}
    {% endfor %}
    {% if _visible %}
    <dl class="extraction-list">
      {% for key, value in _visible %}
      {# Plain Jinja autoescape renders any injected markup inert as text
         (e.g. <img onerror=...> becomes &lt;img onerror=...&gt;). Do NOT strip
         tags here: that would delete legitimate scientific text containing
         angle brackets (p<0.05, n>3, IC50 < 5 uM). No |safe anywhere. #}
      <dt>{{ key }}</dt>
      <dd>{{ value }}</dd>
      {% endfor %}
    </dl>
    {% else %}
    <p class="field-note">No extraction fields stored for this paper.</p>
    {% endif %}
  </div>

  {# --- Score decomposition (only when a breakdown exists) ---------------- #}
  {% if breakdown and breakdown.breakdown %}
  <div class="card">
    <h2>Score decomposition</h2>
    <table class="score-breakdown-table">
      <thead>
        <tr><th>Signal</th><th>Score</th></tr>
      </thead>
      <tbody>
        {% for signal, score in breakdown.breakdown.items() %}
        <tr><td>{{ signal }}</td><td>{{ "%.3f" | format(score | float) }}</td></tr>
        {% endfor %}
      </tbody>
    </table>
  </div>
  {% endif %}

  {# --- Related work + knowledge graph (FE2-4: lazy HTMX fragments) -------- #}
  {# Both load AFTER the page paints (hx-trigger="load") so the detail view
     renders instantly; the graph-backed fragments stream in once ready. The DOI
     is embedded literally in the path so the {doi:path} fragment routes match
     (no urlencode on the path — double-encoding "/" → "%2F" breaks the route). #}
  {% if paper.doi %}
  <div id="corpus-related"
       hx-get="/corpus/{{ paper.doi }}/related"
       hx-trigger="load"
       hx-target="this"
       hx-swap="innerHTML">
    <p class="field-note">Loading related work&hellip;</p>
  </div>
  <div id="corpus-graph"
       hx-get="/corpus/{{ paper.doi }}/graph"
       hx-trigger="load"
       hx-target="this"
       hx-swap="innerHTML">
    <p class="field-note">Loading knowledge graph&hellip;</p>
  </div>
  {% else %}
  <div id="corpus-related"></div>
  <div id="corpus-graph"></div>
  {% endif %}

  {# --- Actions (mirror _partials/paper_card.html P8 idiom) --------------- #}
  {% if paper.doi %}
  {#
    DOI is embedded literally in the URL path — no urlencode — so the
    {doi:path} route (which accepts "10.x/y" verbatim) matches. Double-encoding
    would yield "%2F" in the path and break the route.
  #}
  <div class="card action-row" id="corpus-actions">
    <h2>Actions</h2>
    <button type="button" class="btn btn-secondary btn-sm"
            hx-post="/api/papers/{{ paper.doi }}/relink"
            hx-target="#corpus-actions .action-result"
            hx-swap="innerHTML">Relink</button>
    <button type="button" class="btn btn-secondary btn-sm"
            hx-post="/api/papers/{{ paper.doi }}/re-extract"
            hx-target="#corpus-actions .action-result"
            hx-swap="innerHTML">Re-extract</button>
    <span class="action-result"></span>
  </div>
  {% endif %}
</section>
{% endblock %}