Redirecting the workspace to another location, as shown in the readme, doesn't work

[wags1999]

The readme contains these instructions for redirecting the workspace, using ArtifactStorageOptions.WorkspacePathFormatter to override the path:

var workspaceRoot = Path.Combine(Path.GetTempPath(), "markitdown", "workspaces");

var options = new MarkItDownOptions
{
    ArtifactStorage = ArtifactStorageOptions.Default with
    {
        WorkspacePathFormatter = name => Path.Combine(workspaceRoot, name),
        DeleteOnDispose = false    // keep the workspace directory after conversion
    },
    SegmentOptions = SegmentOptions.Default with
    {
        Image = SegmentOptions.Default.Image with
        {
            KeepArtifactDirectory = true
        }
    }
};

Directory.CreateDirectory(workspaceRoot);

await using var client = new MarkItDownClient(options);
await using var result = await client.ConvertAsync("policy.pdf");

However, before the call to the WorkspacePathFormatter is invoked on line 36:

markitdown/src/MarkItDown/Conversion/ArtifactWorkspaceFactory.cs

Line 36 in bbdf4a3

var directoryPath = storageOptions.WorkspacePathFormatter?.Invoke(workspaceName)

On lines 27-32, if the StorageFactory was not setup on the options, a default path will be returned:

markitdown/src/MarkItDown/Conversion/ArtifactWorkspaceFactory.cs

Lines 27 to 32 in bbdf4a3

	if (storageFactory is null)
	{
	var localDirectory = BuildLocalWorkspacePath(streamInfo, segmentOptions.Image);
	var deleteOnDispose = storageOptions.DeleteOnDispose && !segmentOptions.Image.KeepArtifactDirectory;
	return ArtifactWorkspace.Create(localDirectory, deleteOnDispose, ensureCreated: true);
	}

Using ManagedCode.MarkItDown 10.0.6

[wags1999]

After source code spelunking, the right way to override the temp directory that MarkItDown uses is:

var options = new MarkItDownOptions
{
    RootPath = workspaceRoot
};

or in your Program.cs:

builder.Services.AddMarkItDown().UseRootPath(workspaceRoot);

Would be great if the Readme pointed at these options.

[edgett]

I made #28 to resolve this.

[KSemenenko]

Just merged! Thanks a lot!

[KSemenenko]

Please check latest version and let me know

[wags1999]

Looks great! Thanks!