Skip to content

[any order] Offer a downloadable file only where the client can produce one - #38

Open
fiharet34 wants to merge 1 commit into
joomcode:mainfrom
fiharet34:fix/conditional-download-offers
Open

fiharet34 wants to merge 1 commit into
joomcode:mainfrom
fiharet34:fix/conditional-download-offers

Conversation

@fiharet34

Copy link
Copy Markdown
Contributor

Six skills promise the user a downloadable .csv / .xlsx unconditionally — "This table is the deliverable — and offer it as a downloadable file". Whether that is possible depends entirely on the client. Where it is not, the skill instructs the assistant to promise a file it cannot deliver, and the user is left waiting for a download that never arrives.

The offer is now conditional:

Where the client can produce files, also offer it as a downloadable .csv / .xlsx so the user can save it and bring it back next period as the baseline; where it cannot, the markdown table stands on its own — never offer a download you cannot deliver.

Nothing changes for a client that can produce files. For one that cannot, the markdown table is stated as the deliverable in its own right — which matters most for the tracker skills, since the saved table is what the user pastes back next period for a comparison.

Found running these skills in a deployment with no file output, where every run ended by offering a spreadsheet that could not exist.

Six skills promise the user a downloadable .csv / .xlsx unconditionally —
"This table is the deliverable — and offer it as a downloadable file".
Whether that is possible depends entirely on the client. Where it is not,
the skill instructs the assistant to promise a file it cannot deliver, and
the user waits for a download that never arrives.

The offer is now conditional: where the client can produce files, offer the
spreadsheet as before; where it cannot, the markdown table stands on its own,
and no download is offered. That matters most for the tracker skills, whose
saved table is what the user pastes back next period for a comparison.

Nothing changes for a client that can produce files.

Found running these skills in a deployment with no file output, where every
run ended by offering a spreadsheet that could not exist.
@fiharet34
fiharet34 force-pushed the fix/conditional-download-offers branch from aace0ed to 14c5aa0 Compare September 11, 2026 11:05
@fiharet34
fiharet34 marked this pull request as ready for review September 14, 2026 12:58
@fiharet34 fiharet34 changed the title Offer a downloadable file only where the client can produce one [any order] Offer a downloadable file only where the client can produce one Sep 15, 2026
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