Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@crates/cli/src/outline/model.rs`:
- Around line 52-66: SourcePosition is missing the required byte offset field;
add a pub(super) byte: usize field to the struct, update the doc comment to
mention zero-based byte offset, and modify the impl SourcePosition::new
signature to accept and set the byte value; ensure the serde rename_all remains
camelCase so "byte" serializes correctly and then update any call sites that
construct SourcePosition (e.g., usages of SourcePosition::new or direct struct
literals) to provide the byte argument.
- Around line 285-312: The test serializes_outline_contract is missing
assertions that the OutlineItem.range serializes to the expected
SourceRange/SourcePosition JSON shape; update the test
(serializes_outline_contract) to assert the structure and values of
json["range"] (and nested start/end positions) to match the spec used by
SourcePosition and SourceRange after their update—locate the test and add
assertions verifying keys and numeric fields (e.g., start.line, start.character,
end.line, end.character) and their expected values consistent with test_range().
- Around line 68-86: The JSON shape is wrong: currently SourceRange serializes
byte_offset as an object while the contract expects each position to include its
byte offset. Update the model so SourcePosition includes a byte: usize field
(and derive Serialize/Deserialize), remove the SourceRange.byte_offset field,
and adjust SourceRange::new signature to accept start/end SourcePosition
(containing their byte values); ensure serde rename_all stays camelCase so
range.start.byte and range.end.byte match the documented shape and update any
callers constructing SourceRange accordingly.

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@crates/cli/src/outline/model.rs`:
- Around line 201-205: OutlineMember should preserve an unknown/undetermined
publicness instead of always emitting false; change the field in the
OutlineMember struct from "is_public: bool" to "is_public: Option<bool>" and
annotate it with serde(skip_serializing_if = "Option::is_none") (and keep
#[serde(flatten)] on entry). Update any code that constructs OutlineMember
(e.g., places creating OutlineMember instances or tests) to pass
Some(true)/Some(false) when known or None when unknown, and update any
deserialization/usage sites to handle Option<bool> instead of bool.