chore: sync to main

[rongquan1]

Summary by CodeRabbit

Release Notes

• New Features

  • Documents now track and display expiration status
  • Warning banner alerts users to obfuscated documents

• Improvements

  • Enhanced error handling for unrecognized wallet errors
  • Refined visual styling for expired document indicators
  • Updated asset management interface for clearer expiration display

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

The pull request introduces document expiration detection and updates component rendering to display expiration states. A new utility function getIsExpired checks document expiration dates, the useVerify hook tracks expiration status, and this state is threaded through VerifySection → VerifyResult → AssetManagementApplication. Additionally, a new ObfuscatedMessage component displays warnings for obfuscated documents, and the asset management expired indicator switches from TagBordered to Tag component.

Changes

Cohort / File(s)	Summary
Expiration Tracking & Utilities src/utils/helper.ts, src/components/home/VerifySection/useVerify.ts, src/components/home/VerifySection/VerifySection.tsx	Added getIsExpired utility to detect document expiration; useVerify hook now tracks and returns isExpired state; VerifySection reads and forwards isExpired to VerifyResult.
VerifyResult & ObfuscatedMessage src/components/home/VerifySection/VerifyResult.tsx, src/components/home/VerifySection/ObfuscatedMessage.tsx	VerifyResult accepts optional isExpired prop and passes it to AssetManagementApplication; new ObfuscatedMessage component checks document obfuscation status and displays informational banner when detected.
Asset Management Expired Indicator src/components/AssetManagementPanel/AssetManagementApplication/index.tsx, src/components/AssetManagementPanel/AssetManagementForm/AssetManagementForm.tsx, src/components/AssetManagementPanel/AssetManagementForm/FormVariants/ActionSelectionForm/ActionSelectionForm.tsx	Switched from TagBordered to Tag component for expired indicator; made isExpired prop optional in AssetManagementIsTransferableDocumentProps; removed isExpired prop from ActionSelectionForm (now unused in that context).
Test Updates src/components/AssetManagementPanel/AssetManagementApplication/AssetManagementApplication.test.tsx, src/components/home/VerifySection/VerifyResult.test.tsx, src/components/home/VerifySection/VerifySection.test.tsx	Updated mocks and assertions to reflect component changes: Tag instead of TagBordered, AssetManagementApplication minimal mock to avoid context failures, useVerify mock includes isExpired field.
Error Handling Simplification src/hooks/useContractFunctionHook.tsx, src/hooks/useContractFunctionHook.test.tsx	Removed numeric error code -32603 mapping and changed fallback error handling to return empty string instead of Error.message for unrecognized errors.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• feat: transaction component #28 — Updates AssetManagementApplication component rendering of the expired badge (switching TagBordered → Tag) and handles isExpired prop changes.
• feat: add document renderer with template tabs and attachments #11 — Adds or expands utilities in src/utils/helper.ts like getOpenAttestationData, which getIsExpired depends on.
• feat: add endorsement chain implementation #9 — Modifies the useVerify hook and UseVerifyReturn interface, directly overlapping with the hook's expiration state additions.

Suggested reviewers

• RishabhS7
• manishdex25

Poem

🐰 A document's expiration time is here,
getIsExpired makes the status clear,
Through hooks and props the state will flow,
ObfuscatedMessage warns when docs hide so,
The Tag now shines with clarity's light! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title "chore: sync to main" is vague and generic, failing to convey what was actually changed in the pull request despite substantial modifications across multiple components.	Consider using a more descriptive title that summarizes the main changes, such as "feat: add expired document detection and improve asset management UI" or similar.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
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 docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch develop

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Review rate limit: 0/1 reviews remaining, refill in 60 minutes.}

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