Repository navigation
docs: 태그 기반 글 찾기와 자산 단일 원본화 - #65
Merged
Merged
Conversation
Validate explicit sample publication mappings and serve exact virtual page assets through MkDocs without publishing sample directories. Remove 57 page-side duplicates while retaining all canonical sources and existing used_by ownership. Cover catalog, links, bytes, search, I/O errors, and the full SRE sample suite. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Reject overlapping publication paths against other targets, physical files, and canonical outputs. Explicitly include declared hidden assets while excluding sample sources, and prevent raw Markdown downloads from accepting document-directory shorthand. Add red-first catalog, real build, and link regressions for all three review findings. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Replace ambiguous tags with controlled goal and subject groups. Keep matching documents grouped by canonical topic, combine facets with AND semantics, and preserve legacy discovery URLs as search-excluded redirects. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Use repeated text parameters for Explore filters and preserve Material-owned q values when updating or resetting facets. Cover real Material deep-link and q cleanup behavior, URL persistence, and reload consistency without adding dependencies. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Reject dot-prefixed public path components before generation while retaining hidden sample source payloads. Remove the hidden-target inclusion override and replace its promise with catalog/build rejection tests plus an archive-aligned regression over every current published asset. Preserve the official Pages artifact safety exclusions and all 57 existing mappings. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Owner
Author
자동 코드 리뷰 결과전체 브랜치 diff를 독립 고성능 리뷰 에이전트로 반복 검토했습니다. 리뷰 과정에서 다음 문제를 발견하고 수정했습니다.
최종 재리뷰 결과: Critical/Important 이슈 없음, PR/merge 준비 완료. 최종 검증:
|
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
One or more issues must be addressed before approval.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
This PR unifies document discovery under Explore, centralizes sample-published assets, and enforces the new taxonomy and publishing contracts.
Changes:
- Adds publish-asset validation, copying, collision checks, and sample exclusion.
- Replaces legacy tag/article indexes with taxonomy-driven Explore filtering.
- Migrates document tags and expands validation and integration tests.
File summaries
| File | Description |
|---|---|
| tests/docs/test_validators.py | Updated as part of this pull request. |
| tests/docs/test_topics.py | Updated as part of this pull request. |
| tests/docs/test_site_pipeline.py | Updated as part of this pull request. |
| tests/docs/test_reader_navigation.py | Updated as part of this pull request. |
| tests/docs/test_pages_artifact.py | Updated as part of this pull request. |
| tests/docs/test_indexes.py | Updated as part of this pull request. |
| tests/docs/explore.test.cjs | Updated as part of this pull request. |
| tests/docs/explore-material.test.cjs | Updated as part of this pull request. |
| tests/docs/explore-fixture.cjs | Updated as part of this pull request. |
| scripts/docs/validate_search_index.py | Updated as part of this pull request. |
| scripts/docs/validate_metadata.py | Updated as part of this pull request. |
| scripts/docs/validate_links.py | Updated as part of this pull request. |
| scripts/docs/topics.py | Updated as part of this pull request. |
| scripts/docs/hooks.py | Updated as part of this pull request. |
| scripts/docs/generate_indexes.py | Updated as part of this pull request. |
| scripts/docs/explore.py | Updated as part of this pull request. |
| scripts/docs/content.py | Updated as part of this pull request. |
| README.md | Updated as part of this pull request. |
| mkdocs.yml | Updated as part of this pull request. |
| docs/services/microsoft-foundry/gpt-memory-layer/samples/architecture/sample.yml | Updated as part of this pull request. |
| docs/services/microsoft-foundry/gpt-memory-layer/index.md | Updated as part of this pull request. |
| docs/services/microsoft-foundry/foundry-local-air-gapped/index.md | Updated as part of this pull request. |
| docs/services/microsoft-foundry/codex-closed-network/index.md | Updated as part of this pull request. |
| docs/services/microsoft-foundry/agent-memory/taxonomy/index.md | Updated as part of this pull request. |
| docs/services/microsoft-foundry/agent-memory/taxonomy/images/01-memory-taxonomy.svg | Updated as part of this pull request. |
| docs/services/microsoft-foundry/agent-memory/samples/research-artifacts/sample.yml | Updated as part of this pull request. |
| docs/services/microsoft-foundry/agent-memory/production-evaluation/index.md | Updated as part of this pull request. |
| docs/services/microsoft-foundry/agent-memory/production-evaluation/images/04-production-tiers.svg | Updated as part of this pull request. |
| docs/services/microsoft-foundry/agent-memory/pipeline-retrieval/index.md | Updated as part of this pull request. |
| docs/services/microsoft-foundry/agent-memory/index.md | Updated as part of this pull request. |
| docs/services/microsoft-foundry/agent-memory/frameworks/index.md | Updated as part of this pull request. |
| docs/services/microsoft-foundry/agent-memory/commerce/index.md | Updated as part of this pull request. |
| docs/services/microsoft-foundry/agent-memory/architecture-patterns/index.md | Updated as part of this pull request. |
| docs/services/microsoft-foundry/agent-framework-2026/index.md | Updated as part of this pull request. |
| docs/services/azure-storage/mobile-resumable-upload-tus/samples/spring-application/sample.yml | Updated as part of this pull request. |
| docs/services/azure-storage/mobile-resumable-upload-tus/index.md | Updated as part of this pull request. |
| docs/services/azure-openai/adaptive-ptu-load-balancing/samples/runbooks/sample.yml | Updated as part of this pull request. |
| docs/services/azure-openai/adaptive-ptu-load-balancing/index.md | Updated as part of this pull request. |
| docs/services/azure-monitor/hdinsight-kafka-monitoring/samples/monitoring-assets/sample.yml | Updated as part of this pull request. |
| docs/services/azure-monitor/hdinsight-kafka-monitoring/prometheus-grafana/index.md | Updated as part of this pull request. |
| docs/services/azure-monitor/hdinsight-kafka-monitoring/index.md | Updated as part of this pull request. |
| docs/services/azure-monitor/hdinsight-kafka-monitoring/catch-up-benchmark/index.md | Updated as part of this pull request. |
| docs/services/azure-monitor/dynamic-thresholds-brief/index.md | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/validation-results/index.md | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/setup/index.md | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/scenario-latency/index.md | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/scenario-http-500/index.md | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/scenario-blob-permission/index.md | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/samples/event-lab/scripts/tests/test_repo_readme.py | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/samples/event-lab/scripts/tests/test_published_layout.py | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/samples/event-lab/scripts/tests/test_privacy.py | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/samples/event-lab/scripts/tests/test_briefing_docs.py | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/samples/event-lab/scripts/tests/published_layout.py | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/samples/event-lab/sample.yml | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/results/index.md | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/index.md | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/incident-runbook/index.md | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/images/run-modes-comparison.svg | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/images/root-cause-analysis.svg | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/images/permission-flow.svg | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/images/notification-paths.svg | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/images/memory-unified-search.svg | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/images/memory-auto-learning.svg | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/images/knowledge-sources.svg | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/images/incident-response-flow.svg | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/images/incident-platform-flow.svg | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/images/diagnose-azure-services.svg | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/images/custom-skill-flow.svg | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/images/agent-reasoning-flow.svg | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/event-lab/index.md | Updated as part of this pull request. |
| docs/services/azure-monitor/azure-sre-agent/dynamic-thresholds/index.md | Updated as part of this pull request. |
| docs/services/azure-monitor/aks-private-opentelemetry/index.md | Updated as part of this pull request. |
| docs/services/azure-managed-redis/cluster-failover-recovery/index.md | Updated as part of this pull request. |
| docs/services/azure-load-testing/locust-appgw-aks-private/index.md | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/workload-identity-databricks/index.md | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/spot-h100-kaito/index.md | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/remote-cluster-local-development/index.md | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/python-memory-leak-memray/samples/memray-leak-profiling/sample.yml | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/python-memory-leak-memray/index.md | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/pyroscope-anf-s3/samples/pyroscope-chat-profiling/sample.yml | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/pyroscope-anf-s3/index.md | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/pod-scheduling-agent-pools/index.md | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/pod-database-query-latency/samples/observed-results/sample.yml | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/pod-database-query-latency/index.md | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/pod-affinity-distribution/index.md | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/netapp-files-cpu-io-wait/index.md | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/kaito-open-source-model/index.md | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/file-io-throttling/index.md | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/cni-overlay-nsg/index.md | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/authorization-troubleshooting/index.md | Updated as part of this pull request. |
| docs/services/azure-kubernetes-service/argocd-image-updater-acr/index.md | Updated as part of this pull request. |
| docs/services/azure-database-for-mysql/nodejs-read-write-routing/index.md | Updated as part of this pull request. |
| docs/services/azure-database-for-mysql/blue-green-upgrade/index.md | Updated as part of this pull request. |
| docs/services/azure-cosmos-db/point-read-optimization/index.md | Updated as part of this pull request. |
| docs/services/azure-cosmos-db/nodejs-dns-lookup-bottleneck/index.md | Updated as part of this pull request. |
| docs/services/azure-cosmos-db/nodejs-client-optimization/index.md | Updated as part of this pull request. |
| docs/services/azure-automation/portal-cli-limitations/index.md | Updated as part of this pull request. |
| docs/services/azure-architecture/response-time-optimization/index.md | Updated as part of this pull request. |
| docs/services/azure-application-gateway/waf-path-ip-allowlist/index.md | Updated as part of this pull request. |
| docs/services/azure-application-gateway/sse-response-buffering/index.md | Updated as part of this pull request. |
| docs/services/azure-ai-search/korean-analyzer-comparison/samples/analyzer-comparison/sample.yml | Updated as part of this pull request. |
| docs/services/azure-ai-search/korean-analyzer-comparison/index.md | Updated as part of this pull request. |
| docs/services/azure-ai-search/eventual-consistency-reindex/index.md | Updated as part of this pull request. |
| docs/services/azure-ai-search/custom-vectorization/samples/implementation/sample.yml | Updated as part of this pull request. |
| docs/services/azure-ai-search/custom-vectorization/rag-chunking/index.md | Updated as part of this pull request. |
| docs/services/azure-ai-search/custom-vectorization/index.md | Updated as part of this pull request. |
| docs/services/azure-ai-search/custom-vectorization/gpu-vllm/index.md | Updated as part of this pull request. |
| docs/services/azure-ai-search/custom-vectorization/custom-web-api/index.md | Updated as part of this pull request. |
| docs/services/azure-ai-search/custom-vectorization/bge-m3-vs-qwen3/index.md | Updated as part of this pull request. |
| docs/services/azure-ai-search/custom-vectorization/a10-vs-t4/index.md | Updated as part of this pull request. |
| docs/services/application-development/nodejs-file-io-cpu/index.md | Updated as part of this pull request. |
| docs/index.md | Updated as part of this pull request. |
| docs/contributing/research-template.md | Updated as part of this pull request. |
| docs/contributing/lab-template.md | Updated as part of this pull request. |
| docs/contributing/index.md | Updated as part of this pull request. |
| docs/contributing/guide-template.md | Updated as part of this pull request. |
| docs/contributing/case-template.md | Updated as part of this pull request. |
| docs/assets/stylesheets/extra.css | Updated as part of this pull request. |
| docs/assets/javascripts/explore.js | Updated as part of this pull request. |
| docs/.nav.yml | Updated as part of this pull request. |
| docs-taxonomy.yml | Updated as part of this pull request. |
| CONTRIBUTING.md | Updated as part of this pull request. |
| AGENTS.md | Updated as part of this pull request. |
Review details
- Files reviewed: 110/167 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| errors.append(f"tag_groups.{slug}.tags must be a non-empty list of strings") | ||
| continue | ||
| for tag in values: | ||
| if tag not in tags: |
Comment on lines
+184
to
+188
| path = PurePosixPath(raw) | ||
| if path.is_absolute() or PureWindowsPath(raw).drive or "\\" in raw or ".." in path.parts: | ||
| errors.append( | ||
| f"{relative}: publish {field} must be a relative path without '..' or backslashes: {raw}" | ||
| ) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
변경 내용
sample.yml.publish기반 단일 원본으로 전환했습니다.태그별 보기와전체 글을 하나의글 찾기화면으로 통합했습니다.q와 글 찾기text상태를 분리했습니다./tags/, 21개 태그 URL과/articles/은 검색 제외 redirect로 유지합니다.배경
기존 태그는 작업 목적, 품질 속성, 기술 영역이 섞여 있었고 별도 태그 목록과 전체 글의 차이도 불명확했습니다. 또한 페이지 렌더링 이미지와 샘플 아티팩트 사이에 동일 파일이 57쌍 존재했습니다. 태그를 실제 독자 목적과 핵심 주제로 정리하고, 한 찾기 화면에서 정확한 문서 단위 필터링을 제공하며, 자산은 샘플을 단일 원본으로 사용합니다.
검증
AI agents + 운영: 3개 주제 / 10개 문서AI agents + 운영 + 최적화: 2개 주제 / 2개 문서q닫기 및 reload 후 Exploretext상태 유지리뷰 참고
핵심 로직은 다음 파일에 있습니다.
scripts/docs/topics.py: 게시 자산 및 태그 계약 검증scripts/docs/explore.py: 찾기 페이지 데이터 생성docs/assets/javascripts/explore.js: 클라이언트 필터 상태scripts/docs/generate_indexes.py: 자산·redirect·찾기 페이지 생성