test: fail copy that says staking is live or TrustShell says VETO - #208
DealAppSeo wants to merge 1 commit into
Conversation
The guard reads the landing, /start, and the README. A negation such as staking is not live stays clear.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Strix Security ReviewNo security issues found. Review summaryThe PR adds a single Vitest unit test (tests/slogan.test.ts) that reads a hardcoded list of static copy files and checks their lines against three fixed regular expressions. The file paths are constant and joined from __dirname, and the input is local file content rather than user-controlled data, so there is no path traversal or injection surface. The regular expressions are simple literal alternations with no nested quantifiers, so there is no ReDoS risk. No secrets, authentication, authorization, deserialization, or other security-sensitive logic is introduced. No security issues found in the changed code. Updated for Reviewed by Strix |
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
The PR adds a single Vitest unit test (tests/slogan.test.ts) that reads a hardcoded list of static copy files and checks their lines against three fixed regular expressions. The file paths are constant and joined from __dirname, and the input is local file content rather than user-controlled data, so there is no path traversal or injection surface. The regular expressions are simple literal alternations with no nested quantifiers, so there is no ReDoS risk. No secrets, authentication, authorization, deserialization, or other security-sensitive logic is introduced. No security issues found in the changed code.
Reviewed by Strix
Configure security review settings
tests/slogan.test.ts fails when landing, /start, or README copy says staking is live, or TrustShell says VETO, as a slogan. A negation on the same line stays clear.
Related to #191.