Skip to content

Test: Vale auto-fix workflow v3#575

Closed
jth-nw wants to merge 3 commits intodevfrom
test/vale-autofix-v3
Closed

Test: Vale auto-fix workflow v3#575
jth-nw wants to merge 3 commits intodevfrom
test/vale-autofix-v3

Conversation

@jth-nw
Copy link
Collaborator

@jth-nw jth-nw commented Mar 19, 2026

Summary

  • Test PR to validate the Vale auto-fix workflow with an approved VALE_TOKEN
  • Contains intentional Vale violations across both Phase 1 (script) and Phase 2 (AI) categories
  • Code block section should remain untouched

Expected behavior

  1. Phase 1 commit fixes ~25 mechanical issues (substitutions, contractions, removals, spacing, plurals)
  2. Phase 2 commit fixes ~5 AI-judgment issues (first person, weak link text, once usage, note that, allows you to)
  3. Summary comment posted with fix counts by category
  4. Code block content unchanged

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jth-nw jth-nw requested a review from a team as a code owner March 19, 2026 23:53
@github-actions
Copy link
Contributor

Vale Auto-Fix Summary

9 issues fixed, 0 skipped across 1 files

Category Fixes
AllowsYouTo (rewrite) 1
BoilerplateCrossRef (rewrite) 1
FirstPersonPlural (rewrite) 1
HeadingPunctuation (rewrite) 3
NoteThat (rewrite) 1
OnceUsage (rewrite) 1
WeakLinkText (rewrite) 1

Ask @claude on this PR if you'd like an explanation of any fix.

@github-actions
Copy link
Contributor

Documentation PR Review

Editorial Review

This file is a Vale auto-fix test page containing intentional violations. The following issues are present in the added content beyond what Vale/Dale would catch:

  • First person — Line 39: "We recommend that our users configure the monitoring plan before deployment." uses prohibited first person ("We", "our"). Suggested fix: "Configure the monitoring plan before deployment." or "Users should configure the monitoring plan before deployment."
  • Impersonal phrasing — Line 47: "The system allows you to configure multiple connections." — "allows you to" is a weak, indirect construction. Suggested fix: "You can configure multiple connections."
  • Ambiguous reference — Line 29: "The feature is available in the latest release." — "the feature" and "the latest release" have no antecedent in context. Suggested fix: Name the feature and the specific release.
  • Incorrect usage — Line 58: "ensure to set up the configuration" — "ensure to" is not idiomatic. Suggested fix: "ensure you set up the configuration" or "make sure you set up the configuration."
  • Connector word — Line 43: "Once the installation is complete, verify the service is running." — "Once" is ambiguous (can mean "as soon as" or "after"). Suggested fix: "After the installation is complete, verify the service is running."

Note: This is a test file with intentional violations. Many of the issues above are deliberate and expected to be caught and fixed by the Vale auto-fix workflow. The editorial issues listed here are those that linters are less likely to flag automatically.

Summary

0 Dale issues, 5 editorial suggestions across 1 file.


What to do next:
Comment @claude on this PR followed by your instructions. For example:

  • @claude fix all issues
  • @claude fix only the Dale issues
  • @claude reorganize the prerequisites section
  • @claude help improve the flow of this document

Automated fixes are only available for branches in this repository, not forks.

@github-actions
Copy link
Contributor

Documentation PR Review

Editorial Review

This file is a test artifact with intentional Vale violations — that context is clearly stated in the page itself. Most of the content issues are by design. That said, a few editorial problems exist beyond the intentional test violations:

  • Clarity — Line 19: "The system doesn't require it and can't process it without authorization." The pronoun "it" in the second clause is ambiguous — "it" could refer to "this setting" or to something related to "authorization." Suggested fix: "The system doesn't require this setting and can't process requests without authorization."
  • Clarity — Line 23: "If you want to log in to the portal, the system can handle authentication." The two clauses are logically disconnected — the sentence implies the system handles authentication only conditionally, but the relationship is unclear. Suggested fix: "To log in to the portal, the system handles authentication automatically."
  • Clarity — Line 27: "The system lets you export data." This sentence has no logical connection to the preceding sentence about configuration setup. It reads as a non-sequitur, which may confuse readers even in a test context.
  • Structure — This test file is located at docs/accessanalyzer/12.0/vale-test.md with sidebar_position: 999, placing it in production documentation. Consider whether this belongs in the versioned product docs or should live outside the published doc tree to avoid appearing in the shipped site.

Summary

0 Dale issues (see inline comments), 4 editorial suggestions across 1 file.


What to do next:
Comment @claude on this PR followed by your instructions. For example:

  • @claude fix all issues
  • @claude fix only the Dale issues
  • @claude reorganize the prerequisites section
  • @claude help improve the flow of this document

Automated fixes are only available for branches in this repository, not forks.

@jth-nw jth-nw closed this Mar 20, 2026
@jth-nw jth-nw deleted the test/vale-autofix-v3 branch March 20, 2026 19:19
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