fix(wiki,ingest): P83 — wiki 호출이 만든 codex/claude 세션 self-ingest 루프 차단 (#82)

[hang-in]

Summary

Issue #82 (dicebattle 보고) 의 wiki self-ingest 루프 차단.

• wiki::WIKI_INVOCATION_MARKER (<!-- secall:wiki-update -->) 상수 신설.
• wiki/codex.rs + wiki/claude.rs 의 generate() 가 prompt 앞에 marker prefix.
• ingest::is_noise_session() 에 marker 검출 룰 추가 — 첫 user turn 의 content 에 marker 포함 시 "secall wiki invocation" 사유로 skip.
• 기존 P49 (SECALL_SUMMARY_PROMPT_PREFIX) 와 동일 메커니즘이라 parser 시그니처 변경 없음.
• 신규 unit test 3건 (marker at start / in middle / absent).

Why

secall wiki update → codex exec subprocess → codex CLI 가 ~/.codex/sessions/... 에 새 세션 jsonl 생성 → secall sync 가 그 세션을 일반 사용자 세션과 동일하게 ingest → 다음 wiki 실행 시 그 세션도 분석 대상에 포함 → 무한 wiki 재생성 / 중복 항목. claude 백엔드 (claude -p) 도 동일.

다른 백엔드 (haiku, ollama, lmstudio) 는 HTTP API 라 영향 없음.

조사 결과 기존 ingest/mod.rs::is_noise_session() (P49) 이 이미 tmpdir cwd + SECALL_SUMMARY_PROMPT_PREFIX 검출로 self-ingest 패턴을 차단 중. wiki marker 룰 한 줄 추가만으로 동일 메커니즘 확장 가능.

Test plan

• cargo fmt --all -- --check clean
• cargo clippy --workspace --all-targets -- -D warnings clean
• cargo test -p secall-core --lib ingest:: 110 passed (신규 3건 포함)
• cargo test --workspace --no-fail-fast all green (lib 421 + integration)

Out of scope

본 PR 머지 전 이미 ingest 된 wiki invocation 세션은 marker 가 없음. 사용자가 secall list --recent + secall archive <id> 또는 ~/.codex/sessions/ 직접 삭제로 수동 정리. 자동 일괄 정리는 별도 fast-follow PR 검토.

Closes #82

🤖 Generated with Claude Code

[gemini-code-assist]

Code Review

This pull request implements a fix for issue #82 to prevent self-ingest loops during wiki generation by introducing a WIKI_INVOCATION_MARKER. This marker is prepended to prompts in the Claude and Codex backends, and the is_noise_session function is updated to detect it and skip these sessions. Documentation and unit tests for this feature are also included. A review comment suggests improving the consistency of the is_noise_session function by using the trimmed content variable for both the summary prompt and wiki invocation checks.

[gemini-code-assist]

is_noise_session 함수 내에서 SECALL_SUMMARY_PROMPT_PREFIX는 starts_with를 사용하고, WIKI_INVOCATION_MARKER는 contains를 사용하는 등 검사 방식에 차이가 있습니다.

이미 상단에서 content_trimmed를 계산했으므로 두 번째 조건문에서도 이를 활용하여 일관성을 유지하는 것이 좋으며, 만약 백엔드가 시스템 프롬프트를 앞에 붙이는 경우를 대비해 contains를 사용하신 것이라면 SECALL_SUMMARY_PROMPT_PREFIX 또한 동일한 로직을 적용하는 것을 고려해 보시기 바랍니다.

Suggested change

	let content_trimmed = first_user.content.trim_start();
	if content_trimmed.starts_with(SECALL_SUMMARY_PROMPT_PREFIX) {
	return Some("secall summary prompt");
	}
	if first_user
	.content
	.trim_start()
	.starts_with(SECALL_SUMMARY_PROMPT_PREFIX)
	.contains(crate::wiki::WIKI_INVOCATION_MARKER)
	{
	return Some("secall summary prompt");
	return Some("secall wiki invocation");
	}
	let content_trimmed = first_user.content.trim_start();
	if content_trimmed.starts_with(SECALL_SUMMARY_PROMPT_PREFIX) {
	return Some("secall summary prompt");
	}
	if content_trimmed.contains(crate::wiki::WIKI_INVOCATION_MARKER) {
	return Some("secall wiki invocation");
	}

[hang-in]

Gemini 리뷰 1건 반영 (commit e90e0da) — is_noise_session() 의 두 번째 검사도 content_trimmed 재사용해 일관성 유지. main 머지로 발생한 docs/plans/index.md conflict 도 함께 해소 (P82 → P83 순서 보존). rebase + force-with-lease push.