Skip to content

Revise README content and badge information#112

Merged
chrismaz11 merged 3 commits intomasterfrom
chrismaz11-patch-3
Apr 11, 2026
Merged

Revise README content and badge information#112
chrismaz11 merged 3 commits intomasterfrom
chrismaz11-patch-3

Conversation

@chrismaz11
Copy link
Copy Markdown
Collaborator

Updated README to refine project description and remove redundant badges.

Summary

  • Describe the change

AI Disclosure (optional)

  • AI-assisted changes are included in this PR

Review Checklist

  • Human review requested
  • Tests added or updated where appropriate
  • No secrets, tokens, cookies, or raw PII were added to code, logs, fixtures, or docs
  • Security impact and remaining risks are described

Updated README to refine project description and remove redundant badges.
Copilot AI review requested due to automatic review settings April 10, 2026 02:17
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 10, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
trustsignal Ready Ready Preview, Comment Apr 10, 2026 3:37pm

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR refreshes the top-level README to better describe TrustSignal’s purpose, streamline onboarding instructions, and simplify badge/link presentation for readers evaluating the project.

Changes:

  • Refines the project description and reorganizes sections (problem statement, how it works, API surface).
  • Updates onboarding content (demo, local dev setup, verification curl examples) and adds a repository structure overview.
  • Simplifies badges and adds a TypeScript version badge.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@chrismaz11
Copy link
Copy Markdown
Collaborator Author

@copilot apply changes based on the comments in this thread

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 10, 2026

@copilot apply changes based on the comments in this thread

The vulnerability reporting address was already updated to security@trustsignal.dev in commit 8bc404c, aligning with SECURITY.md. Both changes from the review thread are in place.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 5 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +41 to +43
| Endpoint | Purpose |
|---|---|
| `POST /api/v1/verify` | Submit a verification request |
Copy link

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Markdown table header/rows start with ||, which creates an unintended empty first column and can render the table incorrectly. Use a single leading/trailing | per row (e.g., | Endpoint | Purpose |).

Copilot uses AI. Check for mistakes.
Comment on lines +128 to +131
| Layer | Technology |
|---|---|
| API Server | TypeScript, Fastify v5, Prisma ORM |
| Database | PostgreSQL |
Copy link

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This Tech Stack table uses || at the start of each row, which introduces an empty first column and breaks expected Markdown table formatting. Replace the double leading pipes with a single | consistently across the table.

Copilot uses AI. Check for mistakes.
Comment on lines +143 to +146
| Layer | What Stays in Place |
|---|---|
| Evidence collection | Your platform (Vanta, Drata, internal collector) |
| System of record | Unchanged |
Copy link

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Integration Fit table rows begin with ||, which adds an extra empty column and may render misaligned in Markdown viewers. Adjust the table rows to use a single leading |.

Copilot uses AI. Check for mistakes.
Comment on lines +172 to +175
| Resource | Path |
|---|---|
| Partner Evaluation Kit | [docs/partner-eval/overview.md](docs/partner-eval/overview.md) |
| Evaluator Quickstart | [docs/partner-eval/quickstart.md](docs/partner-eval/quickstart.md) |
Copy link

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Documentation table is formatted with || at the start of each row, creating an unintended blank column. Use standard Markdown table syntax with single leading/trailing pipes so the table renders as two columns.

Copilot uses AI. Check for mistakes.
Comment on lines +206 to +209
| Repository | Purpose |
|---|---|
| [v0-signal-new](https://github.com/TrustSignal-dev/v0-signal-new) | Public website — trustsignal.dev |
| [TrustSignal-App](https://github.com/TrustSignal-dev/TrustSignal-App) | GitHub App for CI verification |
Copy link

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Ecosystem table rows start with ||, which results in an extra empty column and can break table rendering. Use standard Markdown table formatting (single leading |) for the header and all rows.

Copilot uses AI. Check for mistakes.
@chrismaz11 chrismaz11 enabled auto-merge April 11, 2026 03:24
@chrismaz11 chrismaz11 merged commit 67670eb into master Apr 11, 2026
19 of 21 checks passed
@chrismaz11 chrismaz11 deleted the chrismaz11-patch-3 branch April 11, 2026 03:35
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.

3 participants