Skip to content

feat: implement extensible exporter system#78

Merged
maxchampoux merged 6 commits into
WellApp-ai:mainfrom
Muskan244:feature/exporters-system
Aug 10, 2025
Merged

feat: implement extensible exporter system#78
maxchampoux merged 6 commits into
WellApp-ai:mainfrom
Muskan244:feature/exporters-system

Conversation

@Muskan244

Copy link
Copy Markdown
Contributor

Implements a exporter system for invoice data with support for multiple accounting system formats.

Key Features

  • Plugin Architecture: Easy to add new export formats
  • Multiple Formats: JSON, CSV, XML, UBL, QuickBooks, Xero
  • Validation: Built-in data validation
  • Documentation: Comprehensive usage and contribution guides

Changes

  • Added base exporter with validation
  • Implemented exporter factory with registration system
  • Created individual exporters for each format
  • Added unit tests with 100% coverage
  • Updated documentation

Testing

  • All tests pass
  • Verified all export formats
  • Tested with sample data

Closes #27

Muskan244 and others added 3 commits July 30, 2025 01:40
- Add plugin-based exporter architecture
- Implement multiple export formats (JSON, CSV, XML, UBL, QuickBooks, Xero)
- Add base exporter with validation
- Create exporter factory with registration system
- Add comprehensive tests
- Update documentation with usage examples
- Add plugin-based exporter architecture
- Implement multiple export formats (JSON, CSV, XML, UBL, QuickBooks, Xero)
- Add base exporter with validation
- Create exporter factory with registration system
- Add comprehensive tests
- Update documentation with usage examples
Fixes WellApp-ai#27
@Muskan244

Copy link
Copy Markdown
Contributor Author

@maxchampoux Please take a look. Thanks!

@maxchampoux

Copy link
Copy Markdown
Contributor

Hey thanks!
Can you please look at Biome Lint & Format Check / Run Biome Lint & Format (pull_request) - Failing after 7s

@Muskan244

Copy link
Copy Markdown
Contributor Author

Hey, Biome lint and format issues have been fixed. Please take a look and let me know if any other changes are required. Thanks!

@maxchampoux

Copy link
Copy Markdown
Contributor

Hello, not on my end, it's still failing: Biome Lint & Format Check / Run Biome Lint & Format (pull_request)Failing after 7s

@Muskan244

Copy link
Copy Markdown
Contributor Author

Hey, I have made some changes. Please take a look and if it's still failing can you tell me where the problem is. Thanks!

@maxchampoux

Copy link
Copy Markdown
Contributor

Here’s exactly what you can do to resolve the job failure:

Open ai-invoice-extractor/package.json on branch refs/pull/78/head.
Find this line in the devDependencies section:
JSON
"@rollup/rollup-win32-x64-msvc": "^4.45.1",
Remove that line entirely (including any dangling commas if necessary).

Save, commit, and push the change to your branch.
This will fix the CI failure on Linux runners caused by attempting to install a Windows-only package.

@Muskan244

Copy link
Copy Markdown
Contributor Author

Hey, I've removed the Windows-only package from devDependencies. It should work properly now. Thanks for the clear guidance!

@Muskan244

Copy link
Copy Markdown
Contributor Author

@maxchampoux Could you please take a look when you have time? Happy to clarify or make any changes if needed. Thanks!

@maxchampoux maxchampoux merged commit e7e3c03 into WellApp-ai:main Aug 10, 2025
1 of 2 checks passed
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.

Support Multiple Output Formats Compatible with Accounting Systems

2 participants