Skip to content

Document reconnecting shared databases on startup - #671

Open
koppor wants to merge 6 commits into
sqldatabase-postgresfrom
remember-shared-dbs
Open

Document reconnecting shared databases on startup#671
koppor wants to merge 6 commits into
sqldatabase-postgresfrom
remember-shared-dbs

Conversation

@koppor

@koppor koppor commented Sep 1, 2026

Copy link
Copy Markdown
Member

🤖 Low-risk documentation update: mentions that shared databases still connected at quit are reconnected on the next start (JabRef PR JabRef/jabref#16801). Stacked on #670.

🤖 Generated with Claude Code

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

PR Summary by Qodo

Document automatic reconnection of shared databases

📝 Documentation 🕐 Less than 5 minutes

Grey Divider

AI Description

• Document automatic shared-database reconnection after restarting JabRef.
• Clarify that reconnection requires the startup library preference.
High-Level Assessment

The inline clarification is optimal because it places the startup behavior immediately after the shared-database connection instructions. Separate documentation or additional diagrams would add unnecessary complexity for this small behavioral note.

Files changed (1) +2 / -0

Documentation (1) +2 / -0
README.mdExplain shared-database reconnection on startup +2/-0

Explain shared-database reconnection on startup

• Clarifies that shared databases left connected at shutdown reopen automatically on the next start when “Open last edited libraries on startup” is enabled.

en/collaborative-work/sqldatabase/README.md

@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented Sep 1, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0) 🎨 UX issues (0) 🔗 Cross-repo conflicts (0) 📜 Skill insights (0)

Grey Divider


Remediation recommended

1. Reconnection prerequisite omitted ✓ Resolved 🐞 Bug ≡ Correctness
Description
The sentence claims every still-connected shared database reconnects automatically, but databases
requiring authentication cannot do so when the optional **Remember password** setting was not
selected. Users may therefore expect unattended reconnection when JabRef no longer has the
credentials needed to authenticate.
Code

en/collaborative-work/sqldatabase/README.md[27]

+Shared databases that are still connected when you quit JabRef are reconnected automatically on the next start, just like local libraries (when **Open last edited libraries on startup** is enabled in the preferences).
Evidence
The added line makes the startup preference the only stated condition for automatic reconnection,
while the connection instructions establish that saving the database password is optional and
unavailable without a credential store. Thus a password-protected database may lack the
authentication data needed for the promised automatic reconnection.

en/collaborative-work/sqldatabase/README.md[11-11]
en/collaborative-work/sqldatabase/README.md[27-27]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The documentation promises automatic reconnection without explaining that password-protected databases require credentials retained through **Remember password**.
## Issue Context
The same guide says password persistence is optional and may be unavailable when no operating-system credential store can be reached. Describe automatic reconnection as conditional on the necessary credentials being available, or explain that user interaction may otherwise be required.
## Fix Focus Areas
- en/collaborative-work/sqldatabase/README.md[11-11]
- en/collaborative-work/sqldatabase/README.md[27-27]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Tip of the day
💡 Did you know, you can describe a rule in plain language on the Rules page and Qodo drafts it for you

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

Comment thread en/collaborative-work/sqldatabase/README.md Outdated
Base automatically changed from shared-db-url-paste to sqldatabase-postgres September 7, 2026 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant