Slightly improved the invalid name failure message

jamesbraza · jamesbraza · commit f19c8a1ad9dd · 2025-08-06T13:18:20.000-07:00
diff --git a/src/paperqa/types.py b/src/paperqa/types.py
@@ -859,11 +859,12 @@ def populate_bibtex_key_citation(cls, data: dict[str, Any]) -> dict[str, Any]:
                     existing_entry = next(
                         iter(Parser().parse_string(data["bibtex"]).entries.values())
                     )
-                except (PybtexSyntaxError, InvalidNameString):
+                except (PybtexSyntaxError, InvalidNameString) as exc:
                     # InvalidNameString: names like "Kyriacos, Κυριάκος, Athanasiou, Αθανασίου"
                     logger.warning(
                         f"Failed to parse bibtex for DOI {data.get('doi')},"
-                        f" title {data.get('title')}, and bibtex {data['bibtex']}."
+                        f" title {data.get('title')!r}, and bibtex {data['bibtex']}."
+                        f" Failure message: {exc!r}"
                     )
                     existing_entry = None
 

Original file line number	Diff line number	Diff line change
`@@ -859,11 +859,12 @@ def populate_bibtex_key_citation(cls, data: dict[str, Any]) -> dict[str, Any]:`
`859`	`859`	`existing_entry = next(`
`860`	`860`	`iter(Parser().parse_string(data["bibtex"]).entries.values())`
`861`	`861`	`)`
`862`		`- except (PybtexSyntaxError, InvalidNameString):`
	`862`	`+ except (PybtexSyntaxError, InvalidNameString) as exc:`
`863`	`863`	`# InvalidNameString: names like "Kyriacos, Κυριάκος, Athanasiou, Αθανασίου"`
`864`	`864`	`logger.warning(`
`865`	`865`	`f"Failed to parse bibtex for DOI {data.get('doi')},"`
`866`		`- f" title {data.get('title')}, and bibtex {data['bibtex']}."`
	`866`	`+ f" title {data.get('title')!r}, and bibtex {data['bibtex']}."`
	`867`	`+ f" Failure message: {exc!r}"`
`867`	`868`	`)`
`868`	`869`	`existing_entry = None`
`869`	`870`