Skip to content

Conversation

@Pijukatel
Copy link
Contributor

@Pijukatel Pijukatel commented Nov 11, 2025

Description

  • Make sure that storage from ApifyFileSystemStorageClient does not get purged twice due to storage from FileSystemStorageClient pointing to the same location. (Those storage clients will have the same cache key and thus there can be only one.)
  • Ensure that Actor will open input containing KVS on initialization to ensure that an aware storage client is used.
  • Support any possible pre-existing input key and file that is defined through Configuration.input_key. Different input files will have different handling based on their suffix:
    • ".json" is parsed as json.
    • ".txt" is opened as plain text
    • everything else is opened as bytes
    • without extension is tried to be parsed as json first, but falls back to bytes
  • Create a metadata file for the valid pre-existing input file, without modifying the input file (otherwise, cli might detect the change to the input, which would be a false positive)
  • Raise an error if two valid pre-existing input files exist in the expected storage directory.
  • CLI does not respect env variables with the input key so far. TODO: Input file handling apify-cli#960

Issues

Closes: #621
Related to: #INPUT.json Automatically Deleted on Each Run (Python SDK Local Storage Issue)

Testing

  • Added unit tests.
  • Manually tested with [email protected]
    • npx [email protected] run -i {"a":"c"} with pre-existing input file or without input and multiple times in a row
    • npx [email protected] run with pre-existing input file or without input and multiple times in a row

- no modifications to the input file
- accepting various name forms
- error, if multiple valid input files at the ssame time
@github-actions github-actions bot added this to the 127th sprint - Tooling team milestone Nov 11, 2025
@github-actions github-actions bot added t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics. labels Nov 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Actor input via CLI is not reloaded on the second run

2 participants