feat(build): replace alex with direct retext dependencies - #2818
Open
Jamie Kim (jkim323) wants to merge 12 commits into
Open
feat(build): replace alex with direct retext dependencies#2818Jamie Kim (jkim323) wants to merge 12 commits into
Jamie Kim (jkim323) wants to merge 12 commits into
Conversation
added 3 commits
August 31, 2026 11:27
🧹 - Generated by Copilot Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 82416c1e-d4c0-43ab-97a0-ef06656b5ea4
Contributor
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF Scorecard
Scanned Files
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2818 +/- ##
==========================================
+ Coverage 83.14% 83.27% +0.12%
==========================================
Files 183 184 +1
Lines 34039 34499 +460
Branches 27 27
==========================================
+ Hits 28302 28729 +427
- Misses 5734 5767 +33
Partials 3 3
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Contributor
Eval Execution✅ Status: Passed
No changed AI artifacts required evaluation. |
Jamie Kim (jkim323)
marked this pull request as ready for review
September 3, 2026 05:33
Jamie Kim (jkim323)
marked this pull request as draft
September 3, 2026 05:54
Collaborator
Author
|
Copilot resolve the merge conflicts in this pull request |
…ndency-warning # Conflicts: # docs/contributing/evals-ci.md # scripts/README.md Co-authored-by: jkim323 <46459431+jkim323@users.noreply.github.com>
Contributor
Jamie Kim (jkim323)
marked this pull request as ready for review
September 6, 2026 06:02
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.
Description
This change removed the direct
alex@11.0.1development dependency in favorof direct retext dependencies.
retext-equality@6.6.0andvfile-sort@4.0.0developmentdependencies and regenerated the root lockfile.
alex.text()with separate reusable equality and profanityprocessors. Both use
retext-englishandretext-stringify; only theequality processor registers
retext-equality, and only the profanityprocessor registers
retext-profanities.sureness: 1, a confidence threshold ratherthan a severity scale. Rating-0 matches are excluded with and without
-FailOnAlex. Each non-allowlisted rating-1 or rating-2 finding is emittedonce under
source: retext-profanitiesand remains an error.source: alexas the equality report-source alias, not a packagedependency. Equality findings are warnings by default;
-FailOnAlexpromotes only emitted equality findings to errors. It does not change
the profanity threshold or restore excluded matches.
compareMessagebefore allowlist filteringand normalization, retaining the equality-before-profanity source-block
order rather than a global position sort.
profanity findings, separate source blocks, paired phrase allowlists, and
rating-0 exclusion in both
-FailOnAlexmodes.retext implementation and its threshold and compatibility policy.
The JSON report fields, CLI names, and phrase allowlist were retained, but
the historical finding set is intentionally different: profanity findings
are no longer duplicated under
source: alex, and rating-0 profanity matchesare no longer reported. Full historical output equivalence is not claimed.
Related Issue(s)
Closes #2628
Type of Change
Select all that apply:
Code & Documentation:
Infrastructure & Configuration:
AI Artifacts:
hve-builderand addressed all actionable findings.github/instructions/*.instructions.md).github/prompts/*.prompt.md).github/agents/*.agent.md).github/skills/*/SKILL.md).github/hooks/*/*.json)evals/)Other:
.ps1,.sh,.py)Sample Prompts (for AI Artifact Contributions)
User Request:
Execution Flow:
Output Artifacts:
Success Indicators:
For detailed contribution requirements, see:
Testing
npm cipassed after the final merge.npm run test:ps -- -TestPath scripts/tests/evals/Test-EvalSpecText.Tests.ps1passed with 10 tests, 0 failures, and 1 informational skip.
e822f3bdf8e0bc816e996ef1e0d8343b1aa1c12dhas not completed; installation of that snapshot's locked dependencies encountered TLS failures. Module import checks are not a completed locked install or corpus comparison.npm run lint:public-dependency-feedspassed with zero violations.npm run lint:dependency-pinningpassed with zero violations.npm run lint:tables,npm run lint:frontmatter, andnpm run validate:devcontainer-lockfilepassed.npm run validate:docspassed after restoring the independent Docusauruspackage root.
npm run spell-checkpassed.npm run validate:localdid not complete because markdown link validationreceived non-success responses from external URLs in unchanged files.
npm run lint:md-linksdid not complete for the same unchanged externallinks.
npm run ci:eval:lint:textis pending hosted CI.Checklist
Required Checks
AI Artifact Contributions
hve-builderreview mode to review contributionhve-builderreviewRequired Local Checks
The following local-safe validation commands must pass before merging:
npm run validate:localnpm run validate:docsnpm run spell-checknpm run lint:md-linksSecurity Considerations
Additional Notes
The repository template was used.
prebuild-install@7.1.3remains throughunrelated upstream dependencies and is outside this change. Hosted CI must
complete the text lint lane before merge.