Codex/analysis integrity fixes - #26
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
OpenLongevity Pull Request Review Template
Project author: CIPRIAN ȘTEFAN PLEȘCA — cercetător român independent.
Summary
Describe the behavior and scientific scope of this change. State what changed, why it changed, and which part of the platform is affected: evidence model, provider ingestion, persistence, API, dashboard, documentation, security, infrastructure, tests, or release process.
If the change is documentation-only, state whether it updates current implementation facts, target protocols, release history, or limitations. If the change modifies behavior, describe the before and after behavior in plain language.
Scientific Scope
Explain what this pull request does and does not claim. OpenLongevity is a research-navigation and reproducibility platform. A pull request must not introduce unsupported medical advice, diagnosis, treatment recommendations, or proof of human longevity benefit. If the change touches biomarkers, aging clocks, interventions, survival analysis, safety signals, or clinical-trial records, include the interpretation boundary.
Examples of acceptable scoped language:
Examples of language that needs revision:
Evidence and Provenance
If the pull request changes evidence records, ingestion, scoring, graph relationships, gap detection, or exports, describe how provenance is preserved. Identify source identifiers, retrieval timestamps, parser versions, review states, synthetic flags, and limitation fields that are affected. Derived outputs should remain linked to source records.
If the change uses fixtures, state that they are synthetic and whether they are excluded from citation-eligible paths. If the change uses live provider data, identify the provider and retrieval boundary. Do not include restricted datasets, private biomedical records, secrets, or full-text articles unless an explicit documented policy permits them.
Implementation Notes
Describe the technical approach at the level a reviewer needs. Mention affected modules, schema changes, migrations, configuration variables, service boundaries, or API contracts. If a method version changes, say so. If a migration is required, identify the migration and rollback considerations. If a dependency changes, explain why it is needed.
Avoid burying scientific behavior inside implementation detail. A change to scoring, review status, provenance, or export eligibility is a scientific behavior change and should be described as such.
Validation
List the checks performed. Include tests, lint, type checks, documentation audit, Mermaid validation, code-example execution, migration checks, or manual review. If a relevant check was not run, explain why. A skipped database test, missing deployment verification, or fixture-only check should be stated honestly.
Suggested validation commands:
Adjust commands to the environment and current tooling. Do not claim a check passed unless it was actually run.
Risk and Limitations
Describe residual risk. A good pull request names what remains uncertain: provider drift, partial source coverage, unreviewed extraction, fixture-only examples, missing external validation, database migration risk, UI interpretation risk, or deployment uncertainty. This section should be concise but candid.
For documentation changes, identify whether the text describes implemented behavior, release history, or future target protocol. For code changes, identify whether users can see a changed output and how the change is bounded.
Release Notes
State whether release notes, changelog, limitations, citation metadata, or audit reports need updates. If this pull request should not affect release notes, say why. If it changes public interpretation, it probably needs release documentation.
Checklist
Reviewer Focus
Reviewers should prioritize scientific integrity, provenance, reproducibility, security, and user interpretation. Ask whether a reader could overstate the change. Ask whether a derived output remains traceable to source evidence. Ask whether a fixture could be mistaken for observation. Ask whether a claim in documentation matches code behavior. Ask whether a public interface preserves the limitation users need at the point of interpretation.
The goal of review is not only to prevent broken code. It is to keep OpenLongevity honest as it grows.
Academic Quality Bar
A pull request that expands documentation should meet the same standard as code: accurate, traceable, testable where possible, and clear about scope. Long text is not automatically academic. Academic quality means the document distinguishes observation from interpretation, implemented behavior from planned protocol, and evidence navigation from clinical conclusion. If a paragraph makes a strong claim, it should either point to implementation evidence, source provenance, or a clearly labeled future requirement.
Documentation should preserve the author's identity and the independent nature of the project without implying institutional affiliation. References to external standards, providers, or research sources should not imply endorsement. Diagrams should explain actual structure or intended protocols, not decorate weak claims.
Merge Readiness
Before merge, reviewers should be able to answer five questions. What changed? What evidence supports the claim? What checks ran? What risks remain? What user interpretation could be affected? If those questions cannot be answered from the pull request, the change is not yet ready even if the diff is syntactically correct.
When the pull request is part of release preparation, include the release version, tag target, audit status, and any known unfinished gates. Do not publish a release note that suggests a gate passed when it was skipped, unavailable, or only partially checked.
Final Reviewer Note
Review should leave the project easier to trust. If a change is ambitious, make the ambition legible. If a feature is early, say it is early. If a result is synthetic, say it is synthetic. If evidence is unreviewed, preserve that status. The best OpenLongevity pull requests should make future researchers feel that the project is careful with uncertainty as well as capable with code.
Project author: CIPRIAN ȘTEFAN PLEȘCA — cercetător român independent.
OpenLongevity · Project repository.