Skip to content

chore: remove old docs - #25

Merged
PaulNewling merged 1 commit into
mainfrom
remove-unneeded-docs
Sep 15, 2026
Merged

PaulNewling merged 1 commit into
mainfrom
remove-unneeded-docs

Conversation

@PaulNewling

@PaulNewling PaulNewling commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator

Greptile Summary

This PR removes two internal documentation files and adds a patch changeset for the root Sequence Properties package.

Important touched terms

  • Spec deviation: A documented difference between the authoritative specification and implemented behavior. The PR removes the consolidated deviation log, although current workflow comments and changelogs still reference several of its entries.
  • Visualization implementation plan: A temporary design and delivery checklist for the scatterplot and histogram panels. The PR removes it after the corresponding implementation landed.
  • Changeset: Release metadata describing which package receives a semantic-version update and why. The PR adds a patch changeset for @platforma-open/milaboratories.sequence-properties.
  • SD-001–SD-013: Identifiers for individual implementation deviations, including allele filtering, region normalization, receptor/chain inference, and visualization behavior. Their consolidated definitions and rationale are removed by this PR.

Confidence Score: 4/5

The code behavior is unaffected, but the documentation cleanup should also repair or relocate the remaining references to the deleted deviation log.

The visualization plan is obsolete and the changeset targets a valid package, but deleting the deviation log creates concrete broken references in active workflow comments and changelogs.

Files Needing Attention: docs/spec-deviations.md

Important Files Changed

Filename Overview
docs/spec-deviations.md Removes the consolidated record of active implementation deviations while several current code comments and changelogs still link to it.
docs/visualizations-plan.md Removes an obsolete implementation plan whose core scatterplot and histogram work has already landed.
.changeset/real-llamas-count.md Adds a valid patch changeset for the published root block package to record the documentation cleanup.

Fix all with Greploop Fix All in Claude Code

Prompt To Fix All With AI
### Issue 1
docs/spec-deviations.md:1
**Deletion Breaks Active References**

Deleting this deviation log leaves active comments in `workflow/src/main.tpl.tengo` and several package changelogs pointing to a file that no longer exists. Maintainers following those references cannot retrieve the rationale for current behavior such as primary-allele filtering, FR4 normalization, receptor inference, and the deferred visualization reference line. Please preserve or relocate the document, or update all remaining references.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Reviews (1): Last reviewed commit: "chore: remove old docs" | Re-trigger Greptile

Signed-off-by: Paul Newling <paulnewling@gmail.com>
@PaulNewling
PaulNewling added this pull request to the merge queue Sep 15, 2026
Merged via the queue into main with commit d56e94f Sep 15, 2026
11 checks passed
@PaulNewling
PaulNewling deleted the remove-unneeded-docs branch September 15, 2026 12: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