Skip to content

docs: reposition README as verification layer (refs #34, #31)#46

Merged
Rahul Dass (rahuldass19) merged 2 commits into
mainfrom
docs/issue-34-readme-scope-positioning
Jun 22, 2026
Merged

docs: reposition README as verification layer (refs #34, #31)#46
Rahul Dass (rahuldass19) merged 2 commits into
mainfrom
docs/issue-34-readme-scope-positioning

Conversation

@rahuldass19

@rahuldass19 Rahul Dass (rahuldass19) commented Jun 22, 2026

Copy link
Copy Markdown
Member

Summary

Repositions the README to communicate QWED-Tax's identity as a deterministic verification layer — not a tax platform, GST compliance suite, or payroll system.

Implements the positioning direction from #34 (Path B) and addresses the overclaim flagged in #31.

What changed

Identity (top of README)

  • Title/subtitle: Deterministic Verification for Payroll, Tax, and ComplianceThe Verification Gate for AI-Generated Tax Decisions
  • Core positioning sentence added directly below title:

    Tax software answers: "What is the correct calculation?"
    QWED-Tax answers: "Should this AI-generated tax decision be allowed to execute?"

  • Comparison table moved from line 78 to directly below the opening (strongest positioning element)
  • Problem statement reframed around AI-agent-era gate (Allow / Block / Unverifiable → Execution)
  • Procedural Accuracy section moved up (key differentiator)

Scope boundaries

  • Status table: ✅ Production Ready🟢 Verification Layer (both US and India)
  • New Non-Goals section added: GSTN integration, GSTR reconciliation, e-Invoicing/IRN, HSN/SAC classification, tax filing, regulatory change tracking, payroll execution, tax calculation engine
  • Status table note links to Non-Goals section

Language fixes

  • What QWED Tax Verification System (broken heading) → Verification Coverage
  • Enterprise-Grade Tax Code Verification EngineDeterministic Tax Verification Layer for AI Agents
  • Accounts Payable (AP) AutomationAccounts Payable Verification
  • secures the entire Procure-to-Pay cycleverifies tax decisions in the Procure-to-Pay cycle
  • Compliance: Blocks Input Tax CreditVerification: Blocks Input Tax Credit
  • Auto-calculates TDS/Retention amountsVerifies TDS/Retention amounts
  • Run compliance checksRun verification checks
  • India (Income Tax / GST / FEMA)India (CBDT / CBIC / FEMA)

Roadmap

  • ERP Integrations (SAP/Oracle)ERP Verification Connectors — verify tax decisions before they enter SAP/Oracle
  • Massive vision for Global Tax VerificationExpanding verification guard coverage to more jurisdictions and tax rules
  • All roadmap items suffixed with Verification Guards or Verification Connectors

Section reorder

  1. Title + positioning → 2. Comparison table → 3. Problem → 4. Architecture → 5. Procedural Accuracy → 6. Real-world failures → 7. Coverage/Status → 8. Non-goals → 9. Guards → 10. Privacy → 11. Installation → 12. SDK → 13. Examples → 14. Roadmap → 15. Related Packages

What did NOT change

  • Architecture Mermaid diagram (unchanged)
  • Real World Failures table (unchanged)
  • All guard descriptions (unchanged)
  • All code examples (unchanged)
  • Fail-closed contract callout (unchanged)
  • Related Packages table (unchanged)
  • Contributing / License sections (unchanged)
  • Badges (unchanged)

Success criteria

A first-time reader should understand within 30 seconds:

  • QWED-Tax is a verifier
  • QWED-Tax is not a tax platform
  • QWED-Tax exists to gate AI-generated tax decisions before execution
  • QWED-Tax follows deterministic verification, not tax calculation

Refs #34, #31

Summary by CodeRabbit

  • Documentation
    • Updated product overview to clarify QWED-Tax's role as a verification gate for AI-generated tax decisions.
    • Expanded verification coverage documentation with comparison tables and flow diagrams.
    • Refined scope documentation to detail supported deterministic rule verification versus out-of-scope compliance activities.
    • Rewrote examples and guidance for procure-to-pay verification workflows.
    • Updated 2026 product roadmap.

- Replace title/subtitle with verifier-first positioning
- Move comparison table directly below opening (strongest identity proof)
- Reframe problem statement around AI-agent-era gate
- Move Procedural Accuracy section up (key differentiator)
- Remove 'Production Ready' status claims → 'Verification Layer'
- Add explicit Non-Goals section (GSTN, GSTR, e-Invoicing, HSN/SAC, filing, etc.)
- Fix broken heading 'What QWED Tax Verification System' → 'Verification Coverage'
- Replace 'Enterprise-Grade Tax Code Verification Engine' → 'Deterministic Tax Verification Layer for AI Agents'
- Accounts Payable: 'Automation → Verification' language
- Roadmap: 'ERP Integrations → ERP Verification Connectors', remove platform language
- Reorder sections: identity → comparison → problem → architecture → procedural accuracy → coverage → non-goals → guards → privacy → install → SDK → examples → roadmap
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@rahuldass19, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 54 minutes and 23 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits.

🚦 How do rate limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan refill rate.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, the refill rate gradually slows as usage increases. The highest same-day bursts are limited more strictly.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 1ec93a8c-feb2-4e07-b99b-5df29781a3d1

📥 Commits

Reviewing files that changed from the base of the PR and between 8003339 and 557b9ad.

📒 Files selected for processing (1)
  • README.md
📝 Walkthrough

Walkthrough

README.md was substantially rewritten to reposition QWED-Tax as a deterministic verification gate between AI agents and execution systems. A comparison table, verification-flow diagram, and TaxPreFlight TypeScript example were added. Scope, coverage, Zero-Data Leakage, India heading, and Roadmap sections were updated.

Changes

README Rewrite

Layer / File(s) Summary
Hero text and problem/comparison framing
README.md
The hero description now frames QWED-Tax as a verification gate referencing Z3 and Decimal math. "Why QWED-Tax?" and "The Problem" sections drop hallucination wording, add a calculate/execute/verify comparison table, and include an end-to-end verification-flow diagram.
Verification coverage, scope boundaries, and India heading
README.md
"Procedural Accuracy" is shortened to three steps. "Verification Coverage" and "What QWED-Tax Is Not" are reorganized to distinguish deterministic claim verification from out-of-scope activities (filing, classification, live integrations). The India subsection heading changes to "CBDT / CBIC / FEMA."
Zero-Data Leakage, TaxPreFlight SDK example, and Roadmap
README.md
Zero-Data Leakage is rewritten to emphasize local execution. The Accounts Payable and TypeScript SDK sections are replaced with a procure-to-pay framing, a TaxPreFlight browser/frontend API example, and a fail-closed contract statement. Roadmap bullets are updated for 2026 plans.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐇 Hoppity-hop through the tax gate we go,
Z3 and Decimal putting on quite a show,
No hallucinations shall pass through today,
TaxPreFlight checks before agents can pay,
The README shines bright with a fail-closed delight! 🌟

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and accurately reflects the main change: repositioning the README to establish QWED-Tax as a verification layer, with specific references to the relevant issues (#34, #31).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/issue-34-readme-scope-positioning

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🧹 Nitpick comments (2)
README.md (2)

29-29: 🧹 Nitpick | 🔵 Trivial | ⚡ Quick win

Fix heading level to match document structure.

Line 29 uses h3 (###) but should be h2 (##) since it is a top-level section. The previous line is a divider (---), and heading levels should increment consistently after the main title (h1).

📝 Proposed fix
-### ⚔️ Why QWED-Tax?
+## ⚔️ Why QWED-Tax?
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@README.md` at line 29, The heading "⚔️ Why QWED-Tax?" is currently using h3
level (###) but should use h2 level (##) to maintain proper document structure
consistency. Since this is a top-level section following the main title, change
the three hash symbols (###) prefix to two hash symbols (##) for this heading to
ensure consistent heading hierarchy throughout the document.

Source: Linters/SAST tools


45-51: 🧹 Nitpick | 🔵 Trivial | ⚡ Quick win

Specify a language for the fenced code block.

The ASCII diagram block does not declare a language. Add text or ascii as the code block language for clarity and proper rendering.

📝 Proposed fix
-```
+```text
 AI-generated tax decision
         ↓
 Deterministic verification (Allow / Block / Unverifiable)
         ↓
 Execution system (Gusto, Avalara, Stripe)
-```
+```
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@README.md` around lines 45 - 51, The fenced code block containing the ASCII
diagram (showing the flow from AI-generated tax decision through deterministic
verification to execution system) is missing a language specifier. Add `text` as
the language identifier to the opening code fence (the line with ```) to
properly declare the code block language for improved rendering and clarity.

Source: Linters/SAST tools

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@README.md`:
- Line 29: The heading "⚔️ Why QWED-Tax?" is currently using h3 level (###) but
should use h2 level (##) to maintain proper document structure consistency.
Since this is a top-level section following the main title, change the three
hash symbols (###) prefix to two hash symbols (##) for this heading to ensure
consistent heading hierarchy throughout the document.
- Around line 45-51: The fenced code block containing the ASCII diagram (showing
the flow from AI-generated tax decision through deterministic verification to
execution system) is missing a language specifier. Add `text` as the language
identifier to the opening code fence (the line with ```) to properly declare the
code block language for improved rendering and clarity.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 935d58b7-a31d-4df5-9d7d-43018fc1b2b3

📥 Commits

Reviewing files that changed from the base of the PR and between dda926b and 8003339.

📒 Files selected for processing (1)
  • README.md

Comment thread README.md Outdated
@greptile-apps

greptile-apps Bot commented Jun 22, 2026

Copy link
Copy Markdown

Greptile Summary

This PR repositions the QWED-Tax README to communicate the product's identity as a deterministic verification gate for AI-generated tax decisions, rather than a tax platform or compliance suite. The rewrite adds a clear non-goals section, fixes terminology ("compliance" → "verification" throughout), and restructures sections for faster orientation.

  • Three previously-flagged rendering bugs are resolved: the "Why QWED-Tax?" comparison table heading is promoted from H3 to H2, the TypeScript alert label is updated from \" Compliance Block: \" to \" Verification Block: \", and the Accounts Payable section is moved outside the unclosed Python code fence so it renders as prose instead of raw code.
  • A new ## 🚫 What QWED-Tax Is Not section explicitly enumerates out-of-scope features (GSTN integration, GSTR filing, IRN generation, HSN/SAC classification, payroll execution) with a cross-link from the Verification Coverage status table.

Confidence Score: 5/5

Documentation-only change; no executable code modified. All three previously flagged rendering bugs are fixed in this revision.

The change touches only README.md. The three concrete rendering defects called out in earlier review rounds — the skipped H3 heading on the comparison table, the stale "Compliance Block" label in the TypeScript example, and the Accounts Payable section trapped inside an unclosed Python code fence — are all resolved here. No logic, API contracts, or SDK behaviour is altered.

No files require special attention.

Important Files Changed

Filename Overview
README.md Documentation-only rewrite repositioning the product; fixes three pre-existing rendering bugs (H3→H2 heading, unclosed code fence, wrong alert label) and adds a Non-Goals section with an internal anchor link.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["🤖 AI Agent\n(tax decision)"] --> B{"🛡️ QWED-Tax\nVerification Gate"}
    B -->|"Allow"| C["🚀 Execution System\n(Gusto / Avalara / Stripe)"]
    B -->|"Block"| D["🚫 Stop & Throw Error"]
    B -->|"Unverifiable"| E["⚠️ Escalate / Human Review"]

    subgraph "Deterministic Guards"
        F["PayrollGuard / ClassificationGuard\n(W-2 vs 1099, FICA limit)"]
        G["CryptoTaxGuard / InvestmentGuard\n(Sec 115BBH, intraday)"]
        H["GSTGuard / RemittanceGuard\n(RCM, LRS, CGST/SGST/IGST)"]
        I["NexusGuard / ReciprocityGuard\n(economic nexus thresholds)"]
    end

    B --> F & G & H & I
    F & G & H & I -->|"Audit Result"| B

    style B fill:#00C853,stroke:#333,stroke-width:2px,color:white
    style D fill:#ff4444,stroke:#333,stroke-width:2px,color:white
    style E fill:#ff9800,stroke:#333,stroke-width:2px,color:white
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A["🤖 AI Agent\n(tax decision)"] --> B{"🛡️ QWED-Tax\nVerification Gate"}
    B -->|"Allow"| C["🚀 Execution System\n(Gusto / Avalara / Stripe)"]
    B -->|"Block"| D["🚫 Stop & Throw Error"]
    B -->|"Unverifiable"| E["⚠️ Escalate / Human Review"]

    subgraph "Deterministic Guards"
        F["PayrollGuard / ClassificationGuard\n(W-2 vs 1099, FICA limit)"]
        G["CryptoTaxGuard / InvestmentGuard\n(Sec 115BBH, intraday)"]
        H["GSTGuard / RemittanceGuard\n(RCM, LRS, CGST/SGST/IGST)"]
        I["NexusGuard / ReciprocityGuard\n(economic nexus thresholds)"]
    end

    B --> F & G & H & I
    F & G & H & I -->|"Audit Result"| B

    style B fill:#00C853,stroke:#333,stroke-width:2px,color:white
    style D fill:#ff4444,stroke:#333,stroke-width:2px,color:white
    style E fill:#ff9800,stroke:#333,stroke-width:2px,color:white
Loading

Reviews (2): Last reviewed commit: "fix: address Sentry, CodeRabbit, Greptil..." | Re-trigger Greptile

Comment thread README.md Outdated
Comment thread README.md
- Move closing code fence before Accounts Payable heading (Sentry)
- Fix heading level ### → ## for Why QWED-Tax? (CodeRabbit, Greptile P2)
- Add text language specifier to ASCII diagram code block (CodeRabbit)
- Replace 'Compliance Block' → 'Verification Block' in TS example (Greptile P2)
@sonarqubecloud

Copy link
Copy Markdown

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