change type of cli "embed -c" to be STR not FILE

[mhalle]

The type of the "content" argument was erroneously set to be FILE with a "type=click.Path(...)" argument, which content is a string.

[simonw]

Huh! I'm surprised that didn't result in any test failures. Here's the commit with the mistake: 77cf56e

[simonw]

I think this is the test that exercised that:

llm/tests/test_embed_cli.py

Lines 43 to 58 in 5ba34db

	def test_embed_output_format(tmpdir, format_, expected, scenario):
	runner = CliRunner()
	args = ["embed", "--format", format_, "-m", "embed-demo"]
	input = None
	if scenario == "argument":
	args.extend(["-c", "hello world"])
	elif scenario == "file":
	path = tmpdir / "input.txt"
	path.write_text("hello world", "utf-8")
	args.extend(["-i", str(path)])
	elif scenario == "stdin":
	input = "hello world"
	args.extend(["-i", "-"])
	result = runner.invoke(cli, args, input=input)
	assert result.exit_code == 0
	assert result.output == expected

[simonw]

I figured out why it was mostly working - it's because this just said it had to be a writable path, and most strings are also writable paths:

llm/llm/cli.py

Lines 1039 to 1044 in 5ba34db

	@click.option(
	"-c",
	"--content",
	help="Content to embed",
	type=click.Path(file_okay=True, allow_dash=False, dir_okay=False, writable=True),
	)

But if I do this:

touch no-write
chmod -w no-write
llm embed -c no-write -m ada-002

I get this error:

Usage: llm embed [OPTIONS] [COLLECTION] [ID]
Try 'llm embed --help' for help.

Error: Invalid value for '-c' / '--content': File 'no-write' is not writable.

Which is fixed by this PR.