saitejatirunagari Claude Opus 4.6 commited on
Commit
7ed6a2f
·
1 Parent(s): aaa65ea

fix: bulk history panel overlap + SKILLS section diagnostics (v2.6.3)

Browse files

loadBulkHistory now hides resultsPanel before showing bulkPanel —
prevents single-gen download buttons overlapping bulk download buttons
(user downloads wrong PDF). Added diagnostic logging for SKILLS
section regex failure on container.

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

HISTORY.md CHANGED
@@ -4,6 +4,19 @@ A running log of everything built, fixed, and changed. Most recent first.
4
 
5
  ---
6
 
 
 
 
 
 
 
 
 
 
 
 
 
 
7
  ## 2026-09-02 — Fix root cause: gap weave was consuming placement keywords (v2.6.2)
8
 
9
  **Root cause found:** Step 5.4 (weave) was adding ALL competency gap keywords to
 
4
 
5
  ---
6
 
7
+ ## 2026-09-03 — Fix overlapping panels + SKILLS diagnostics (v2.6.3)
8
+
9
+ - **Bulk history panel overlap fix.** `loadBulkHistory` click handler now hides
10
+ `resultsPanel` before showing `bulkPanel` — prevents single-gen download buttons
11
+ from overlapping with bulk download buttons (user downloads wrong PDF).
12
+ - **SKILLS section diagnostics.** `_add_to_skills_section` now logs context around
13
+ "SKILLS" when the regex fails, and `ats_safe.py` logs whether the SKILLS section
14
+ exists in `final_latex` before calling `place_keywords_naturally`.
15
+ - **`_sanitize_for_tectonic` fix** (from prior session) — `re.sub` lambda replacements
16
+ prevent `bad escape \u` crash during LaTeX compilation.
17
+
18
+ ---
19
+
20
  ## 2026-09-02 — Fix root cause: gap weave was consuming placement keywords (v2.6.2)
21
 
22
  **Root cause found:** Step 5.4 (weave) was adding ALL competency gap keywords to
extension/manifest.json CHANGED
@@ -1,7 +1,7 @@
1
  {
2
  "manifest_version": 3,
3
  "name": "ATS Resume Generator",
4
- "version": "2.6.2",
5
  "description": "Tailors your resume (PDF or LaTeX) to any job posting using the ATS pipeline.",
6
  "permissions": [
7
  "storage",
 
1
  {
2
  "manifest_version": 3,
3
  "name": "ATS Resume Generator",
4
+ "version": "2.6.3",
5
  "description": "Tailors your resume (PDF or LaTeX) to any job posting using the ATS pipeline.",
6
  "permissions": [
7
  "storage",
extension/popup/popup.js CHANGED
@@ -1009,6 +1009,7 @@ async function loadBulkHistory() {
1009
  if (!latest) return;
1010
  bulkResults = latest.results;
1011
  bulkHistoryMode = true;
 
1012
  bulkPanel.classList.add('visible');
1013
  generateBtn.style.display = 'none';
1014
  bulkBtn.style.display = 'none';
 
1009
  if (!latest) return;
1010
  bulkResults = latest.results;
1011
  bulkHistoryMode = true;
1012
+ resultsPanel.classList.remove('visible');
1013
  bulkPanel.classList.add('visible');
1014
  generateBtn.style.display = 'none';
1015
  bulkBtn.style.display = 'none';
resume-tailor-extension-v2.6.3.zip ADDED
Binary file (49.5 kB). View file
 
src/ats_safe.py CHANGED
@@ -365,8 +365,9 @@ def generate_alignment_safe(
365
  and c.get("category", "") in _COMPETENCY_CATS]
366
  _tool_gaps = [c for c in valid if c.get("category") == "tool"
367
  and c.get("normalized_concept") not in _supported_now]
 
368
  print(f"[kw-place] placing {len(placeable)} of {len(valid)} criteria "
369
- f"({len(_tool_gaps)} tool gaps withheld)")
370
  final_latex, kw_recs = place_keywords_naturally(
371
  final_latex, placeable, max_per_section=20, max_per_bullet=10,
372
  dedup_text=resume_text)
 
365
  and c.get("category", "") in _COMPETENCY_CATS]
366
  _tool_gaps = [c for c in valid if c.get("category") == "tool"
367
  and c.get("normalized_concept") not in _supported_now]
368
+ _skills_ok = bool(re.search(r"\\section\*?\{\s*SKILLS\s*\}", final_latex, re.I))
369
  print(f"[kw-place] placing {len(placeable)} of {len(valid)} criteria "
370
+ f"({len(_tool_gaps)} tool gaps withheld) skills_section={_skills_ok} latex_len={len(final_latex)}")
371
  final_latex, kw_recs = place_keywords_naturally(
372
  final_latex, placeable, max_per_section=20, max_per_bullet=10,
373
  dedup_text=resume_text)
src/resume_rewrite.py CHANGED
@@ -1033,7 +1033,11 @@ def _add_to_skills_section(latex_src: str, remaining: List[dict],
1033
  from .latex_resume import latex_to_text
1034
  skills_m = re.search(r"\\section\*?\{\s*SKILLS\s*\}", latex_src, re.I)
1035
  if not skills_m:
1036
- print("[kw-skills] no SKILLS section found")
 
 
 
 
1037
  return latex_src, []
1038
  existing_low = _norm(latex_to_text(latex_src[skills_m.start():]))
1039
  def _wb(pn: str) -> bool:
 
1033
  from .latex_resume import latex_to_text
1034
  skills_m = re.search(r"\\section\*?\{\s*SKILLS\s*\}", latex_src, re.I)
1035
  if not skills_m:
1036
+ _raw_idx = latex_src.find("SKILLS")
1037
+ _ctx = repr(latex_src[max(0,_raw_idx-60):_raw_idx+60]) if _raw_idx >= 0 else "N/A"
1038
+ print(f"[kw-skills] no SKILLS section found (raw 'SKILLS' at idx={_raw_idx}, "
1039
+ f"len={len(latex_src)}, ctx={_ctx}, "
1040
+ f"sections={[m.group() for m in re.finditer(r'section.*?[}]', latex_src[:3000])]})")
1041
  return latex_src, []
1042
  existing_low = _norm(latex_to_text(latex_src[skills_m.start():]))
1043
  def _wb(pn: str) -> bool: