lucy-cli 1.7.0

use std::collections::BTreeMap;
use std::ffi::OsStr;
#[cfg(unix)]
use std::ffi::{CStr, CString, OsString};
use std::fs;
use std::io::{self, Read};
#[cfg(unix)]
use std::os::fd::{AsRawFd, FromRawFd, RawFd};
#[cfg(unix)]
use std::os::unix::ffi::{OsStrExt, OsStringExt};
use std::path::{Component, Path, PathBuf};
use std::process::{Command, Stdio};

use serde::{Deserialize, Serialize};

use crate::config::config_dir;

#[derive(Debug)]
pub struct ContextError(String);

impl ContextError {
    fn new(message: impl Into<String>) -> Self {
        Self(message.into())
    }
}

impl std::fmt::Display for ContextError {
    fn fmt(&self, formatter: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
        formatter.write_str(&self.0)
    }
}

impl std::error::Error for ContextError {}

impl From<io::Error> for ContextError {
    fn from(_error: io::Error) -> Self {
        Self::new("instruction context discovery error")
    }
}

#[derive(Debug, Clone, PartialEq, Eq)]
pub struct InstructionSource {
    pub path: PathBuf,
    pub contents: String,
}

/// A discovered Agent Skill. `contents` is retained so explicit invocations
/// use the exact, symlink-safe snapshot discovered when the session started.
#[derive(Debug, Clone, Serialize, Deserialize, PartialEq, Eq)]
pub struct SkillEntry {
    pub name: String,
    pub description: String,
    pub path: PathBuf,
    #[serde(default)]
    pub contents: String,
    #[serde(default = "default_model_invocable")]
    pub model_invocable: bool,
}

fn default_model_invocable() -> bool {
    true
}

#[derive(Debug, Clone, PartialEq, Eq)]
pub struct BootContext {
    pub system_prompt: String,
    pub cwd: PathBuf,
    pub instruction_files: Vec<InstructionSource>,
    pub skills: Vec<SkillEntry>,
}

#[cfg(test)]
fn resolve_boot_context(
    home: &Path,
    cwd: &Path,
    configured_prompt: &str,
) -> Result<BootContext, ContextError> {
    resolve_boot_context_with_api_key_env(home, cwd, configured_prompt, None)
}

pub(crate) fn resolve_boot_context_with_api_key_env(
    home: &Path,
    cwd: &Path,
    configured_prompt: &str,
    api_key_env: Option<&str>,
) -> Result<BootContext, ContextError> {
    let cwd = fs::canonicalize(cwd)
        .map_err(|_error| ContextError::new("unable to resolve working directory"))?;
    let root = git_root(&cwd, api_key_env);
    let project_directories = ancestor_directories(&root, &cwd);

    let mut instruction_files = Vec::new();
    if let Some(instruction) = preferred_instruction(&config_dir(home))? {
        instruction_files.push(instruction);
    }
    for directory in &project_directories {
        if let Some(instruction) = preferred_instruction(directory)? {
            instruction_files.push(instruction);
        }
    }

    let mut readme_files = Vec::new();
    for directory in &project_directories {
        if let Some(readme) = readme_for_directory(directory)? {
            readme_files.push(readme);
        }
    }

    // More-specific project locations override an earlier skill with the
    // same declared name.
    let mut skills = BTreeMap::new();
    discover_skills(&home.join(".agents").join("skills"), &mut skills)?;
    for directory in &project_directories {
        discover_skills(&directory.join(".agents").join("skills"), &mut skills)?;
    }
    let skills = skills.into_values().collect::<Vec<_>>();
    let system_prompt = build_system_prompt(
        configured_prompt,
        &cwd,
        &instruction_files,
        &readme_files,
        &skills,
    );

    Ok(BootContext {
        system_prompt,
        cwd,
        instruction_files,
        skills,
    })
}

fn git_root(cwd: &Path, api_key_env: Option<&str>) -> PathBuf {
    let mut command = Command::new("git");
    command
        .arg("-C")
        .arg(cwd)
        .args(["rev-parse", "--show-toplevel"]);
    if let Some(api_key_env) = api_key_env
        .map(str::trim)
        .filter(|api_key_env| !api_key_env.is_empty())
    {
        command.env_remove(api_key_env);
    }
    let output = command
        .stdout(Stdio::piped())
        .stderr(Stdio::null())
        .output();

    match output {
        Ok(output) if output.status.success() => {
            let text = String::from_utf8_lossy(&output.stdout).trim().to_owned();
            if !text.is_empty() {
                if let Ok(path) = fs::canonicalize(text) {
                    return path;
                }
            }
            cwd.to_owned()
        }
        _ => cwd.to_owned(),
    }
}

fn ancestor_directories(root: &Path, cwd: &Path) -> Vec<PathBuf> {
    let mut directories = Vec::new();
    let mut current = cwd;
    loop {
        directories.push(current.to_owned());
        if current == root {
            break;
        }
        let Some(parent) = current.parent() else {
            break;
        };
        if !cwd.starts_with(parent) || !parent.starts_with(root) {
            break;
        }
        current = parent;
    }
    directories.reverse();
    directories
}

#[cfg(unix)]
struct ContextDirectory {
    file: fs::File,
}

#[cfg(not(unix))]
struct ContextDirectory {
    path: PathBuf,
}

#[cfg(unix)]
fn path_component_unavailable(error: &io::Error) -> bool {
    error.kind() == io::ErrorKind::NotFound
        || error.raw_os_error() == Some(libc::ENOTDIR)
        || error.raw_os_error() == Some(libc::ELOOP)
}

#[cfg(unix)]
fn open_directory_at(parent: RawFd, name: &OsStr) -> io::Result<Option<fs::File>> {
    let name = CString::new(name.as_bytes())
        .map_err(|_error| io::Error::new(io::ErrorKind::InvalidInput, "path contains NUL"))?;
    let flags = libc::O_RDONLY | libc::O_DIRECTORY | libc::O_NOFOLLOW | libc::O_CLOEXEC;
    let fd = unsafe { libc::openat(parent, name.as_ptr(), flags, 0) };
    if fd < 0 {
        let error = io::Error::last_os_error();
        if path_component_unavailable(&error) {
            return Ok(None);
        }
        return Err(error);
    }
    Ok(Some(unsafe { fs::File::from_raw_fd(fd) }))
}

#[cfg(unix)]
fn open_instruction_file_at(parent: RawFd, name: &OsStr) -> io::Result<Option<fs::File>> {
    let name = CString::new(name.as_bytes())
        .map_err(|_error| io::Error::new(io::ErrorKind::InvalidInput, "path contains NUL"))?;
    let flags = libc::O_RDONLY | libc::O_NONBLOCK | libc::O_CLOEXEC;
    let fd = unsafe { libc::openat(parent, name.as_ptr(), flags, 0) };
    if fd < 0 {
        let error = io::Error::last_os_error();
        if path_component_unavailable(&error) {
            return Ok(None);
        }
        return Err(error);
    }
    let file = unsafe { fs::File::from_raw_fd(fd) };
    if !file.metadata()?.is_file() {
        return Ok(None);
    }
    Ok(Some(file))
}

#[cfg(unix)]
fn open_file_at(parent: RawFd, name: &OsStr) -> io::Result<Option<fs::File>> {
    let name = CString::new(name.as_bytes())
        .map_err(|_error| io::Error::new(io::ErrorKind::InvalidInput, "path contains NUL"))?;
    let flags = libc::O_RDONLY | libc::O_NOFOLLOW | libc::O_NONBLOCK | libc::O_CLOEXEC;
    let fd = unsafe { libc::openat(parent, name.as_ptr(), flags, 0) };
    if fd < 0 {
        let error = io::Error::last_os_error();
        if path_component_unavailable(&error) {
            return Ok(None);
        }
        return Err(error);
    }
    let file = unsafe { fs::File::from_raw_fd(fd) };
    if !file.metadata()?.is_file() {
        return Ok(None);
    }
    Ok(Some(file))
}

#[cfg(unix)]
impl ContextDirectory {
    fn open(path: &Path) -> io::Result<Option<Self>> {
        let start = if path.is_absolute() {
            OsStr::new("/")
        } else {
            OsStr::new(".")
        };
        let Some(file) = open_directory_at(libc::AT_FDCWD, start)? else {
            return Ok(None);
        };
        let mut directory = Self { file };

        for component in path.components() {
            let name = match component {
                Component::Prefix(_) => {
                    return Err(io::Error::new(
                        io::ErrorKind::InvalidInput,
                        "path prefix is not supported on Unix",
                    ));
                }
                Component::RootDir | Component::CurDir => continue,
                Component::ParentDir => OsStr::new(".."),
                Component::Normal(name) => name,
            };
            let Some(file) = open_directory_at(directory.file.as_raw_fd(), name)? else {
                return Ok(None);
            };
            directory = Self { file };
        }

        Ok(Some(directory))
    }

    fn open_child_directory(&self, name: &OsStr) -> io::Result<Option<Self>> {
        let Some(file) = open_directory_at(self.file.as_raw_fd(), name)? else {
            return Ok(None);
        };
        Ok(Some(Self { file }))
    }

    fn open_instruction_file(&self, name: &OsStr) -> io::Result<Option<fs::File>> {
        open_instruction_file_at(self.file.as_raw_fd(), name)
    }

    fn open_regular_file(&self, name: &OsStr) -> io::Result<Option<fs::File>> {
        open_file_at(self.file.as_raw_fd(), name)
    }

    fn entries(&self) -> io::Result<Vec<OsString>> {
        read_directory_entries(&self.file)
    }
}

#[cfg(not(unix))]
impl ContextDirectory {
    fn open(path: &Path) -> io::Result<Option<Self>> {
        match fs::symlink_metadata(path) {
            Ok(metadata) if metadata.file_type().is_symlink() || !metadata.is_dir() => Ok(None),
            Ok(_) => Ok(Some(Self {
                path: path.to_owned(),
            })),
            Err(error) if error.kind() == io::ErrorKind::NotFound => Ok(None),
            Err(error) => Err(error),
        }
    }

    fn open_child_directory(&self, name: &OsStr) -> io::Result<Option<Self>> {
        Self::open(&self.path.join(name))
    }

    fn open_instruction_file(&self, name: &OsStr) -> io::Result<Option<fs::File>> {
        open_instruction_file(&self.path.join(name))
    }

    fn open_regular_file(&self, name: &OsStr) -> io::Result<Option<fs::File>> {
        open_regular_file(&self.path.join(name))
    }

    fn entries(&self) -> io::Result<Vec<std::ffi::OsString>> {
        fs::read_dir(&self.path)?
            .map(|entry| entry.map(|entry| entry.file_name()))
            .collect()
    }
}

#[cfg(unix)]
struct DirectoryStream(*mut libc::DIR);

#[cfg(unix)]
impl Drop for DirectoryStream {
    fn drop(&mut self) {
        unsafe {
            libc::closedir(self.0);
        }
    }
}

#[cfg(unix)]
fn reset_directory_errno() {
    #[cfg(any(target_os = "linux", target_os = "android"))]
    unsafe {
        *libc::__errno_location() = 0;
    }
    #[cfg(any(
        target_os = "macos",
        target_os = "ios",
        target_os = "tvos",
        target_os = "watchos",
        target_os = "freebsd",
        target_os = "dragonfly",
        target_os = "openbsd",
        target_os = "netbsd"
    ))]
    unsafe {
        *libc::__error() = 0;
    }
}

#[cfg(unix)]
fn directory_errno() -> libc::c_int {
    #[cfg(any(target_os = "linux", target_os = "android"))]
    {
        unsafe { *libc::__errno_location() }
    }
    #[cfg(any(
        target_os = "macos",
        target_os = "ios",
        target_os = "tvos",
        target_os = "watchos",
        target_os = "freebsd",
        target_os = "dragonfly",
        target_os = "openbsd",
        target_os = "netbsd"
    ))]
    {
        unsafe { *libc::__error() }
    }
    #[cfg(not(any(
        target_os = "linux",
        target_os = "android",
        target_os = "macos",
        target_os = "ios",
        target_os = "tvos",
        target_os = "watchos",
        target_os = "freebsd",
        target_os = "dragonfly",
        target_os = "openbsd",
        target_os = "netbsd"
    )))]
    {
        0
    }
}

#[cfg(unix)]
fn read_directory_entries(file: &fs::File) -> io::Result<Vec<OsString>> {
    let duplicate = unsafe { libc::dup(file.as_raw_fd()) };
    if duplicate < 0 {
        return Err(io::Error::last_os_error());
    }
    let directory = unsafe { libc::fdopendir(duplicate) };
    if directory.is_null() {
        let error = io::Error::last_os_error();
        unsafe {
            libc::close(duplicate);
        }
        return Err(error);
    }
    let directory = DirectoryStream(directory);
    let mut entries = Vec::new();
    loop {
        reset_directory_errno();
        let entry = unsafe { libc::readdir(directory.0) };
        if entry.is_null() {
            let error_number = directory_errno();
            if error_number != 0 {
                return Err(io::Error::from_raw_os_error(error_number));
            }
            break;
        }
        let name = unsafe { CStr::from_ptr((*entry).d_name.as_ptr()) }.to_bytes();
        if name != b"." && name != b".." {
            entries.push(OsString::from_vec(name.to_vec()));
        }
    }
    Ok(entries)
}

#[cfg(not(unix))]
fn open_instruction_file(path: &Path) -> io::Result<Option<fs::File>> {
    let file = match fs::File::open(path) {
        Ok(file) => file,
        Err(error) if error.kind() == io::ErrorKind::NotFound => return Ok(None),
        Err(error) => return Err(error),
    };
    if !file.metadata()?.is_file() {
        return Ok(None);
    }
    Ok(Some(file))
}

#[cfg(not(unix))]
fn open_regular_file(path: &Path) -> io::Result<Option<fs::File>> {
    let mut options = fs::OpenOptions::new();
    options.read(true);
    match fs::symlink_metadata(path) {
        Ok(metadata) if metadata.file_type().is_symlink() || !metadata.is_file() => {
            return Ok(None);
        }
        Ok(_) => {}
        Err(error) if error.kind() == io::ErrorKind::NotFound => return Ok(None),
        Err(error) => return Err(error),
    }

    let file = match options.open(path) {
        Ok(file) => file,
        Err(error) if error.kind() == io::ErrorKind::NotFound => return Ok(None),
        Err(error) => return Err(error),
    };
    if !file.metadata()?.is_file() {
        return Ok(None);
    }
    Ok(Some(file))
}

fn read_open_file(mut file: fs::File) -> io::Result<String> {
    let mut contents = String::new();
    file.read_to_string(&mut contents)?;
    Ok(contents)
}

fn preferred_instruction(directory: &Path) -> Result<Option<InstructionSource>, ContextError> {
    let Some(directory_fd) = ContextDirectory::open(directory)
        .map_err(|_error| ContextError::new("unable to inspect instruction context"))?
    else {
        return Ok(None);
    };

    for name in [OsStr::new("AGENTS.md"), OsStr::new("CLAUDE.md")] {
        let Some(file) = directory_fd
            .open_instruction_file(name)
            .map_err(|_error| ContextError::new("unable to inspect instruction context"))?
        else {
            continue;
        };
        let contents = read_open_file(file)
            .map_err(|_error| ContextError::new("unable to read instruction context"))?;
        return Ok(Some(InstructionSource {
            path: directory.join(name),
            contents,
        }));
    }
    Ok(None)
}

const README_CHAR_LIMIT: usize = 1000;

fn truncate_readme(contents: &str) -> String {
    let trimmed = contents.trim_end();
    if trimmed.chars().count() <= README_CHAR_LIMIT {
        return trimmed.to_owned();
    }
    let truncated: String = trimmed.chars().take(README_CHAR_LIMIT).collect();
    format!("{truncated}\n\n[README truncated; showing first 1000 characters]")
}

fn readme_for_directory(directory: &Path) -> Result<Option<InstructionSource>, ContextError> {
    let Some(directory_fd) = ContextDirectory::open(directory)
        .map_err(|_error| ContextError::new("unable to inspect instruction context"))?
    else {
        return Ok(None);
    };
    let Some(file) = directory_fd
        .open_instruction_file(OsStr::new("README.md"))
        .map_err(|_error| ContextError::new("unable to inspect instruction context"))?
    else {
        return Ok(None);
    };
    let contents = read_open_file(file)
        .map_err(|_error| ContextError::new("unable to read instruction context"))?;
    Ok(Some(InstructionSource {
        path: directory.join("README.md"),
        contents: truncate_readme(&contents),
    }))
}

fn discover_skills(
    skills_root: &Path,
    skills: &mut BTreeMap<String, SkillEntry>,
) -> Result<(), ContextError> {
    let Some(skills_parent_path) = skills_root.parent() else {
        return Ok(());
    };
    let Some(skills_parent) = ContextDirectory::open(skills_parent_path)
        .map_err(|_error| ContextError::new("unable to inspect skill context"))?
    else {
        return Ok(());
    };
    let Some(skills_name) = skills_root.file_name() else {
        return Ok(());
    };
    let Some(skills_directory) = skills_parent
        .open_child_directory(skills_name)
        .map_err(|_error| ContextError::new("unable to inspect skill context"))?
    else {
        return Ok(());
    };
    discover_skill_directory(skills_root, &skills_directory, skills)
}

fn discover_skill_directory(
    path: &Path,
    directory: &ContextDirectory,
    skills: &mut BTreeMap<String, SkillEntry>,
) -> Result<(), ContextError> {
    if let Some(file) = directory
        .open_regular_file(OsStr::new("SKILL.md"))
        .map_err(|_error| ContextError::new("unable to inspect skill context"))?
    {
        if let Ok(contents) = read_open_file(file) {
            if let Some((name, description, model_invocable)) = parse_skill_frontmatter(&contents) {
                skills.insert(
                    name.clone(),
                    SkillEntry {
                        name,
                        description,
                        path: path.join("SKILL.md"),
                        contents,
                        model_invocable,
                    },
                );
            }
        }
    }

    let mut names = directory
        .entries()
        .map_err(|_error| ContextError::new("unable to inspect skill context"))?;
    names.sort();
    for name in names {
        let Some(child) = directory
            .open_child_directory(&name)
            .map_err(|_error| ContextError::new("unable to inspect skill context"))?
        else {
            continue;
        };
        discover_skill_directory(&path.join(&name), &child, skills)?;
    }
    Ok(())
}

fn parse_skill_frontmatter(contents: &str) -> Option<(String, String, bool)> {
    let lines = contents.lines().collect::<Vec<_>>();
    if lines.first().map(|line| line.trim()) != Some("---") {
        return None;
    }
    let end = lines
        .iter()
        .enumerate()
        .skip(1)
        .find(|(_, line)| line.trim() == "---")
        .map(|(index, _)| index)?;

    let mut name = None;
    let mut description = None;
    let mut model_invocable = true;
    let mut index = 1;
    while index < end {
        let line = lines[index];
        let trimmed = line.trim_start();
        if let Some(value) = trimmed.strip_prefix("name:") {
            name = parse_scalar(value);
            index += 1;
            continue;
        }
        if let Some(value) = trimmed.strip_prefix("disable-model-invocation:") {
            model_invocable = !matches!(value.trim(), "true" | "True" | "TRUE");
            index += 1;
            continue;
        }
        if let Some(value) = trimmed.strip_prefix("description:") {
            let value = value.trim();
            if matches!(value, "|" | "|-" | "|+" | ">" | ">-" | ">+") {
                let folded = value.starts_with('>');
                index += 1;
                let mut block = Vec::new();
                while index < end {
                    let block_line = lines[index];
                    if !block_line.trim().is_empty() && !block_line.starts_with(char::is_whitespace)
                    {
                        break;
                    }
                    block.push(block_line.trim().to_owned());
                    index += 1;
                }
                description = Some(if folded {
                    block.join(" ").trim().to_owned()
                } else {
                    block.join("\n").trim().to_owned()
                });
                continue;
            }
            description = parse_scalar(value);
        }
        index += 1;
    }

    let name = name?.trim().to_owned();
    let description = description?.trim().to_owned();
    if !valid_skill_name(&name) || description.is_empty() || description.chars().count() > 1024 {
        return None;
    }
    Some((name, description, model_invocable))
}

fn valid_skill_name(name: &str) -> bool {
    !name.is_empty()
        && name.len() <= 64
        && !name.starts_with('-')
        && !name.ends_with('-')
        && !name.contains("--")
        && name
            .bytes()
            .all(|byte| byte.is_ascii_lowercase() || byte.is_ascii_digit() || byte == b'-')
}

fn parse_scalar(value: &str) -> Option<String> {
    let value = value.trim();
    if value.is_empty() {
        return None;
    }
    if value.starts_with('"') && value.ends_with('"') && value.len() >= 2 {
        return serde_json::from_str(value).ok();
    }
    if value.starts_with('\'') && value.ends_with('\'') && value.len() >= 2 {
        return Some(value[1..value.len() - 1].replace("''", "'"));
    }
    Some(value.to_owned())
}

/// Keep metadata in the XML-shaped progressive-disclosure catalog from
/// changing its structure. Full skill contents are intentionally not escaped:
/// they are loaded only when a skill is selected as instructions.
fn escape_xml(text: &str) -> String {
    text.replace('&', "&amp;")
        .replace('<', "&lt;")
        .replace('>', "&gt;")
        .replace('\"', "&quot;")
        .replace('\'', "&apos;")
}

fn build_system_prompt(
    configured_prompt: &str,
    cwd: &Path,
    instruction_files: &[InstructionSource],
    readme_files: &[InstructionSource],
    skills: &[SkillEntry],
) -> String {
    let mut sections = vec![configured_prompt.trim_end().to_owned()];
    sections.push(format!("## Working directory\n{}", cwd.display()));
    for instruction in instruction_files {
        sections.push(format!(
            "## Instructions from {}\n{}",
            instruction.path.display(),
            instruction.contents.trim_end()
        ));
    }
    for readme in readme_files {
        sections.push(format!(
            "## README from {}\n{}",
            readme.path.display(),
            readme.contents.trim_end()
        ));
    }
    let invocable_skills = skills
        .iter()
        .filter(|skill| skill.model_invocable)
        .collect::<Vec<_>>();
    if !invocable_skills.is_empty() {
        let mut catalog = String::from("<available_skills>\n");
        for skill in invocable_skills {
            catalog.push_str(&format!(
                "<skill>\n<name>{}</name>\n<description>{}</description>\n<location>{}</location>\n</skill>\n",
                escape_xml(&skill.name),
                escape_xml(&skill.description),
                escape_xml(&skill.path.display().to_string())
            ));
        }
        catalog.push_str("</available_skills>");
        sections.push(catalog);
    }
    sections.join("\n\n")
}

#[cfg(test)]
mod tests {
    use super::*;
    #[cfg(unix)]
    use std::os::unix::fs::symlink;
    use std::sync::atomic::{AtomicU64, Ordering};
    use std::time::{SystemTime, UNIX_EPOCH};

    static TEMP_COUNTER: AtomicU64 = AtomicU64::new(0);

    fn temporary_tree() -> (PathBuf, PathBuf) {
        let home = loop {
            let stamp = SystemTime::now()
                .duration_since(UNIX_EPOCH)
                .expect("clock")
                .as_nanos();
            let counter = TEMP_COUNTER.fetch_add(1, Ordering::Relaxed);
            let path = std::env::temp_dir().join(format!(
                "lucy-context-{stamp}-{}-{counter}",
                std::process::id()
            ));
            match fs::create_dir(&path) {
                Ok(()) => break path,
                Err(error) if error.kind() == io::ErrorKind::AlreadyExists => continue,
                Err(error) => panic!("temp tree: {error}"),
            }
        };
        let home = fs::canonicalize(&home).expect("canonical temp tree");
        let project = home.join("project").join("nested");
        fs::create_dir_all(&project).expect("tree");
        Command::new("git")
            .arg("-C")
            .arg(home.join("project"))
            .args(["init", "-q"])
            .output()
            .expect("git init");
        (home, project)
    }

    #[test]
    fn context_uses_precedence_and_specific_skill_override() {
        let (home, cwd) = temporary_tree();
        let project = home.join("project");
        fs::create_dir_all(config_dir(&home)).expect("global dir");
        fs::write(config_dir(&home).join("CLAUDE.md"), "global claude").expect("global");
        fs::write(config_dir(&home).join("AGENTS.md"), "global agents").expect("global agents");
        fs::write(project.join("CLAUDE.md"), "root claude").expect("root claude");
        fs::write(project.join("AGENTS.md"), "root agents").expect("root agents");
        fs::write(cwd.join("CLAUDE.md"), "nested claude").expect("nested claude");

        let global_skill = home.join(".agents/skills/shared/SKILL.md");
        let root_skill = project.join(".agents/skills/shared/SKILL.md");
        let nested_skill = cwd.join(".agents/skills/nested/SKILL.md");
        fs::create_dir_all(global_skill.parent().expect("parent")).expect("global skills");
        fs::create_dir_all(root_skill.parent().expect("parent")).expect("root skills");
        fs::create_dir_all(nested_skill.parent().expect("parent")).expect("nested skills");
        fs::write(
            global_skill,
            "---\nname: shared\ndescription: global description\n---\n# global",
        )
        .expect("global skill");
        fs::write(
            root_skill,
            "---\nname: shared\ndescription: root description\n---\n# root",
        )
        .expect("root skill");
        fs::write(
            &nested_skill,
            "---\nname: nested\ndescription: nested description\n---\n# nested",
        )
        .expect("nested skill");

        let context = resolve_boot_context(&home, &cwd, "configured").expect("context");
        assert_eq!(context.instruction_files.len(), 3);
        assert_eq!(
            context.instruction_files[0].path,
            config_dir(&home).join("AGENTS.md")
        );
        assert!(context.instruction_files[0]
            .contents
            .contains("global agents"));
        assert!(context.instruction_files[1]
            .contents
            .contains("root agents"));
        assert!(context.instruction_files[2]
            .contents
            .contains("nested claude"));
        assert!(!context.system_prompt.contains("root claude"));
        assert!(context.system_prompt.contains("root description"));
        assert!(!context.system_prompt.contains("global description"));
        assert!(context.system_prompt.contains("nested description"));
        assert!(context
            .system_prompt
            .contains(&nested_skill.display().to_string()));
        assert!(!context.system_prompt.contains("# nested"));
        assert!(context.system_prompt.contains("## Working directory"));
        assert!(context
            .system_prompt
            .contains(&context.cwd.display().to_string()));

        fs::remove_dir_all(home).expect("remove tree");
    }

    #[test]
    fn context_failure_does_not_echo_a_secret_bearing_path() {
        let (home, _cwd) = temporary_tree();
        let missing = home.join("provider-secret-context-missing");
        let error = resolve_boot_context(&home, &missing, "configured")
            .expect_err("missing working directory");
        let message = error.to_string();
        assert!(message.contains("working directory"));
        assert!(!message.contains("provider-secret"));
        assert!(!message.contains(&missing.display().to_string()));
        fs::remove_dir_all(home).expect("remove tree");
    }

    #[cfg(unix)]
    #[test]
    fn context_follows_symlinked_instruction_files_but_ignores_symlinked_skills() {
        let (home, cwd) = temporary_tree();
        let project = home.join("project");
        fs::create_dir_all(config_dir(&home)).expect("global directory");
        let global_instruction_target = home.join("global-instructions.md");
        fs::write(&global_instruction_target, "symlinked global instructions")
            .expect("global target");
        symlink(
            &global_instruction_target,
            config_dir(&home).join("AGENTS.md"),
        )
        .expect("global instruction symlink");
        fs::write(
            config_dir(&home).join("CLAUDE.md"),
            "real global instructions",
        )
        .expect("global fallback");

        let project_instruction_target = home.join("project-instructions.md");
        fs::write(
            &project_instruction_target,
            "symlinked project instructions",
        )
        .expect("project target");
        symlink(&project_instruction_target, project.join("AGENTS.md"))
            .expect("project agents symlink");
        symlink(&project_instruction_target, project.join("CLAUDE.md"))
            .expect("project claude symlink");

        let global_skills = home.join(".agents/skills");
        fs::create_dir_all(&global_skills).expect("global skills");
        let linked_directory_target = home.join("linked-skill-directory");
        fs::create_dir(&linked_directory_target).expect("linked directory target");
        fs::write(
            linked_directory_target.join("SKILL.md"),
            "---\nname: linked-directory\ndescription: linked directory\n---\n",
        )
        .expect("linked directory skill");
        symlink(
            &linked_directory_target,
            global_skills.join("linked-directory"),
        )
        .expect("skill directory symlink");

        let linked_file_target = home.join("linked-skill-file.md");
        fs::write(
            &linked_file_target,
            "---\nname: linked-file\ndescription: linked file\n---\n",
        )
        .expect("linked file target");
        let linked_file_directory = global_skills.join("linked-file");
        fs::create_dir(&linked_file_directory).expect("linked file directory");
        symlink(&linked_file_target, linked_file_directory.join("SKILL.md"))
            .expect("skill file symlink");

        let valid_skill = global_skills.join("valid/SKILL.md");
        fs::create_dir_all(valid_skill.parent().expect("valid skill parent"))
            .expect("valid skill directory");
        fs::write(
            &valid_skill,
            "---\nname: valid\ndescription: valid skill\n---\n",
        )
        .expect("valid skill");

        let project_skill_target = home.join("project-skills");
        let project_skill = project_skill_target.join("root-only/SKILL.md");
        fs::create_dir_all(project_skill.parent().expect("project skill parent"))
            .expect("project skill target");
        fs::write(
            &project_skill,
            "---\nname: project-only\ndescription: project only\n---\n",
        )
        .expect("project skill");
        fs::create_dir_all(project.join(".agents")).expect("project agents directory");
        symlink(&project_skill_target, project.join(".agents/skills")).expect("skill root symlink");

        let context = resolve_boot_context(&home, &cwd, "configured").expect("context");
        assert_eq!(context.instruction_files.len(), 2);
        assert_eq!(
            context.instruction_files[0].path,
            config_dir(&home).join("AGENTS.md")
        );
        assert_eq!(
            context.instruction_files[0].contents,
            "symlinked global instructions"
        );
        assert_eq!(context.instruction_files[1].path, project.join("AGENTS.md"));
        assert_eq!(
            context.instruction_files[1].contents,
            "symlinked project instructions"
        );
        assert_eq!(context.skills.len(), 1);
        assert_eq!(context.skills[0].name, "valid");
        assert!(context
            .system_prompt
            .contains("symlinked global instructions"));
        assert!(context
            .system_prompt
            .contains("symlinked project instructions"));
        assert!(!context.system_prompt.contains("real global instructions"));
        assert!(!context.system_prompt.contains("linked-directory"));
        assert!(!context.system_prompt.contains("linked-file"));
        assert!(!context.system_prompt.contains("project-only"));

        fs::remove_dir_all(home).expect("remove tree");
    }

    #[cfg(unix)]
    #[test]
    fn context_ignores_symlinked_intermediate_parents() {
        let (home, cwd) = temporary_tree();
        let linked_home_target = home.join("linked-home-target");
        fs::create_dir_all(linked_home_target.join(".config/lucy")).expect("linked Lucy directory");
        fs::write(
            linked_home_target.join(".config/lucy/AGENTS.md"),
            "symlinked intermediate instructions",
        )
        .expect("linked instructions");
        let linked_skill = linked_home_target.join(".agents/skills/linked/SKILL.md");
        fs::create_dir_all(linked_skill.parent().expect("linked skill parent"))
            .expect("linked skill directory");
        fs::write(
            &linked_skill,
            "---\nname: linked-intermediate\ndescription: linked intermediate\n---\n",
        )
        .expect("linked skill");
        let linked_home = home.join("linked-home");
        symlink(&linked_home_target, &linked_home).expect("linked home");

        let context = resolve_boot_context(&linked_home, &cwd, "configured").expect("context");
        assert!(context.instruction_files.is_empty());
        assert!(context.skills.is_empty());
        assert!(!context.system_prompt.contains("symlinked intermediate"));
        assert!(!context.system_prompt.contains("linked-intermediate"));

        fs::remove_dir_all(home).expect("remove tree");
    }

    #[test]
    fn skill_frontmatter_enforces_standard_names_and_hides_explicit_only_skills() {
        assert!(
            parse_skill_frontmatter("---\nname: valid-skill-2\ndescription: visible\n---\n")
                .is_some()
        );
        assert!(
            parse_skill_frontmatter("---\nname: Invalid_Skill\ndescription: invalid\n---\n")
                .is_none()
        );
        let hidden = SkillEntry {
            name: "private-skill".to_owned(),
            description: "hidden from automatic selection".to_owned(),
            path: PathBuf::from("/skills/private/SKILL.md"),
            contents: "instructions".to_owned(),
            model_invocable: false,
        };
        let prompt = build_system_prompt("configured", Path::new("/"), &[], &[], &[hidden]);
        assert!(!prompt.contains("private-skill"));
        assert_eq!(escape_xml("a<&>\"'"), "a&lt;&amp;&gt;&quot;&apos;");
    }

    #[test]
    fn invalid_skill_metadata_is_skipped() {
        let (home, cwd) = temporary_tree();
        let invalid = cwd.join(".agents/skills/invalid/SKILL.md");
        fs::create_dir_all(invalid.parent().expect("parent")).expect("skill dir");
        fs::write(invalid, "---\nname: invalid\n---\nbody").expect("skill");
        let context = resolve_boot_context(&home, &cwd, "configured").expect("context");
        assert!(context.skills.is_empty());
        assert!(!context.system_prompt.contains("invalid"));
        fs::remove_dir_all(home).expect("remove tree");
    }

    #[test]
    fn system_prompt_includes_cwd() {
        let (home, cwd) = temporary_tree();
        let context = resolve_boot_context(&home, &cwd, "configured").expect("context");
        assert!(context.system_prompt.contains("## Working directory"));
        assert!(context
            .system_prompt
            .contains(&context.cwd.display().to_string()));
        fs::remove_dir_all(home).expect("remove tree");
    }

    #[test]
    fn readme_full_content_in_system_prompt() {
        let (home, cwd) = temporary_tree();
        fs::write(cwd.join("README.md"), "# Project\n\nShort readme.").expect("readme");
        let context = resolve_boot_context(&home, &cwd, "configured").expect("context");
        assert!(context.system_prompt.contains("## README from"));
        assert!(context.system_prompt.contains("# Project"));
        assert!(context.system_prompt.contains("Short readme."));
        assert!(!context.system_prompt.contains("[README truncated"));
        fs::remove_dir_all(home).expect("remove tree");
    }

    #[test]
    fn readme_truncated_when_too_long() {
        let (home, cwd) = temporary_tree();
        let content = "a".repeat(1000) + "b";
        fs::write(cwd.join("README.md"), &content).expect("readme");
        let context = resolve_boot_context(&home, &cwd, "configured").expect("context");
        assert!(context
            .system_prompt
            .contains("[README truncated; showing first 1000 characters]"));
        fs::remove_dir_all(home).expect("remove tree");
    }

    #[test]
    fn readme_from_multiple_ancestor_directories() {
        let (home, cwd) = temporary_tree();
        let project = home.join("project");
        fs::write(project.join("README.md"), "root readme").expect("root readme");
        fs::write(cwd.join("README.md"), "nested readme").expect("nested readme");
        let context = resolve_boot_context(&home, &cwd, "configured").expect("context");
        assert!(context.system_prompt.contains("root readme"));
        assert!(context.system_prompt.contains("nested readme"));
        fs::remove_dir_all(home).expect("remove tree");
    }

    #[test]
    fn no_readme_works_without_error() {
        let (home, cwd) = temporary_tree();
        let context = resolve_boot_context(&home, &cwd, "configured").expect("context");
        assert!(!context.system_prompt.contains("## README from"));
        fs::remove_dir_all(home).expect("remove tree");
    }
}