docs: add QBO report export domain skill#314
Open
alexph-dev wants to merge 1 commit intobrowser-use:mainfrom
Open
docs: add QBO report export domain skill#314alexph-dev wants to merge 1 commit intobrowser-use:mainfrom
alexph-dev wants to merge 1 commit intobrowser-use:mainfrom
Conversation
Contributor
There was a problem hiding this comment.
1 issue found across 1 file
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="agent-workspace/domain-skills/qbo/report-export.md">
<violation number="1" location="agent-workspace/domain-skills/qbo/report-export.md:84">
P2: Selecting the last blob target from an unordered target list can save a stale/wrong PDF when multiple QBO blob targets exist.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review, or fix all with cubic.
| for t in cdp("Target.getTargets")["targetInfos"] | ||
| if t.get("url", "").startswith("blob:https://qbo.intuit.com/") | ||
| ] | ||
| return blobs[-1] if blobs else None |
Contributor
There was a problem hiding this comment.
P2: Selecting the last blob target from an unordered target list can save a stale/wrong PDF when multiple QBO blob targets exist.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At agent-workspace/domain-skills/qbo/report-export.md, line 84:
<comment>Selecting the last blob target from an unordered target list can save a stale/wrong PDF when multiple QBO blob targets exist.</comment>
<file context>
@@ -0,0 +1,157 @@
+ for t in cdp("Target.getTargets")["targetInfos"]
+ if t.get("url", "").startswith("blob:https://qbo.intuit.com/")
+ ]
+ return blobs[-1] if blobs else None
+
+def save_qbo_pdf_blob(out_path, timeout=30):
</file context>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary\n- add a QuickBooks Online custom-report export domain skill\n- document the clean QBO Export to PDF flow and why Page.printToPDF is the wrong tool\n- document the fast blob-save path for QBO-generated PDF viewer blobs\n- capture compact menu/report-state checks and validation steps\n\n## Notes\n- sanitized to avoid business-specific report names, company names, customer codes, or local paths\n\n## Testing\n- docs-only change; not run
Summary by cubic
Adds a domain skill doc for exporting QuickBooks Online custom reports to clean PDFs using the in-app export and direct blob save path. Clarifies why CDP
Page.printToPDFis wrong and provides simple validation steps for reliable automation.blob:https://qbo.intuit.com/...and save bytes.pdfinfo/pdftotextvalidation.Page.printToPDF, OS save sheets, stale tabs, and premature blob fetches.Written for commit 1583bd7. Summary will update on new commits.