Add additionalContentExclusionPolicies to SessionConfig (public high-level API)
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 65/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- typescript
- 領域
- api
調査の方向性
公開されている SessionConfigBase インターフェースと createSession() および resumeSession() のエントリーポイントから開始し、それらの生成された SessionOpenOptions 型へのマッピングを比較します。既存の低レベルポリシーフィールドがどのように表現され、エクスポートされているかを確認します。公開型がドキュメント化され、作成時と再開時の両方でポリシーがアトミックに転送され、カバレッジによって両方のパスが検証されれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Add additionalContentExclusionPolicies to SessionConfig (public high-level API)
Summary
The Copilot SDK exposes content exclusion policy support at the low-level RPC layer
(SessionOpenOptions.additionalContentExclusionPolicies, session.rpc.options.update,
session.rpc.permissions.configure) but does not surface it through the public
SessionConfig / SessionConfigBase interface used by createSession() and
resumeSession(). This makes it impossible for SDK consumers to set per-session
content exclusion rules through the documented, stable public API.
Motivation
Content exclusion is a core compliance and data-governance feature. At GitHub
organization level, admins configure it via repository settings. However, SDK
consumers — especially those building multi-tenant agentic applications — need to
inject additional exclusion rules per session, beyond what GitHub org settings provide:
- Multi-tenant SaaS: Each tenant has their own data classification rules that
must not be shared with the model context (PII paths, credentials directories,
confidential source trees). - Regulated environments (GxP, HIPAA, SOC 2): Compliance requirements dictate
that certain file patterns are never read by the model, regardless of what
the repository-level GitHub policy says. - Agent isolation: Pipeline orchestrators that spawn per-role sessions need to
scope each session's readable context to only the files that role is authorized
to see, without coupling this to GitHub org settings.
Current workaround (and why it is insufficient)
The only way to set exclusion rules today is via session.rpc.permissions.configure()
after the session is created:
const session = await client.createSession({ ... });
// Race condition: session may have already started reading files before this call
await session.rpc.permissions.configure({
additionalContentExclusionPolicies: [{
rules: [{ paths: ["**/secrets/**", "**/.env*"], source: { name: "tenant-policy", type: "host" } }],
last_updated_at: Date.now(),
scope: "all",
}],
});
Problems:
- Race condition: The session may read files before the
configurecall returns.
There is no way to guarantee exclusion rules are active before the first turn. - Undocumented surface:
session.rpcis a low-level raw RPC accessor marked
@experimental. Nothing in the README or SDK docs points consumers here for
content exclusion. - Inconsistent with the rest of the API: Every other security-relevant session
option (availableTools,excludedTools,onPermissionRequest,
skipCustomInstructions,gitHubToken,skipEmbeddingRetrieval) is set
throughSessionConfigBase. Content exclusion policies should follow the same
pattern. - Mid-session update (
rpc.options.update) is the only per-turn path: There is
no way to supply policies atomically with the session creation RPC.
Proposed change
Add additionalContentExclusionPolicies to SessionConfigBase:
export interface SessionConfigBase {
// ... existing fields ...
/**
* Additional content-exclusion policies to merge into the session's policy set
* on creation (and on resume). Rules are evaluated alongside natively-discovered
* GitHub organization / repository policies.
*
* Use this to inject application-level or tenant-level path exclusions that
* are not covered by GitHub organization settings — for example, in multi-tenant
* deployments where each session must be scoped to its tenant's permitted file set.
*
* Each policy rule that matches a file path will cause the CLI to deny Copilot
* access to that file, equivalent to a `PermissionDecisionDeniedByContentExclusionPolicy`
* outcome on the permission request for that path.
*
* @example
* ```typescript
* const session = await client.createSession({
* additionalContentExclusionPolicies: [{
* scope: "all",
* last_updated_at: Date.now(),
* rules: [
* {
* paths: ["**/.env*", "**/secrets/**", "**/*.pem"],
* source: { name: "tenant-compliance-policy", type: "host" },
* },
* ],
* }],
* });
* ```
*
* @experimental
*/
additionalContentExclusionPolicies?: ContentExclusionPolicy[];
}
The ContentExclusionPolicy / ContentExclusionPolicyRule types should be exported
as first-class public types (not re-exports of the generated RPC types), with clear
JSDoc:
/** Scope for a content exclusion policy. */
export type ContentExclusionPolicyScope = "repo" | "all";
/** A single rule within a content exclusion policy. */
export interface ContentExclusionPolicyRule {
/** Glob patterns for file paths to exclude from Copilot context. */
paths: string[];
/** The rule is active only when at least one of these patterns matches the file content. */
ifAnyMatch?: string[];
/** The rule is active only when none of these patterns matches the file content. */
ifNoneMatch?: string[];
/** Source label identifying who issued this rule. */
source: { name: string; type: string };
}
/** A content exclusion policy applied to a session. */
export interface ContentExclusionPolicy {
/** The rules that make up this policy. */
rules: ContentExclusionPolicyRule[];
/** ISO 8601 timestamp or Unix ms epoch when the policy was last updated. */
last_updated_at: string | number;
/** Whether the policy applies to the current repository only, or all repositories. */
scope: ContentExclusionPolicyScope;
}
Expected behavior
- Policies supplied in
SessionConfig.additionalContentExclusionPoliciesare
applied atomically with the session creation RPC (SessionOpenOptions), before
any tool execution can occur. - On
resumeSession, the same field re-applies the policies so resumed sessions
respect updated tenant rules. - Policies compose with GitHub org-level exclusions (same as the current RPC
behavior): a file is excluded if it matches any policy from any source.
Alternatives considered
- Rely on
onPermissionRequestto deny individual file reads: This works per
permission prompt but does not prevent the model from receiving file content via
attachment or retrieval paths that do not go through the permission system. - Use
availableTools/excludedToolsto remove read tools: Too coarse —
this removes read access entirely rather than scoping it to allowed paths. - GitHub org content exclusion settings: Not available in multi-tenant scenarios
where the SDK host is not a GitHub organization admin, or where tenants are
not GitHub entities at all.
References
- Generated RPC types:
SessionOpenOptions.additionalContentExclusionPolicies,
PermissionsConfigureParams.additionalContentExclusionPolicies,
OptionsUpdateParams.additionalContentExclusionPolicies - Related:
SessionConfigBase.gitHubToken— per-session GitHub identity for
org-level content exclusion discovery - Related:
SessionConfigBase.availableTools/excludedTools— tool-level
filtering, not path-level content filtering
- 主要言語
- Java
- スター
- 10.5k
- フォーク
- 1.5k
- 平均マージ
- 1日 9時間
- マージ済み PR(30日)
- 131
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
github/copilot-sdk のほかの issue
-
agentic-workflows
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
github/copilot-sdk#2709 · コメント 1 件 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
github/copilot-sdk#2673 ·
-
bug testing
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
github/copilot-sdk#2628 ·
-
agentic-workflows
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
github/copilot-sdk#2627 · コメント 1 件 ·
-
agentic-workflows
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
github/copilot-sdk#2493 ·
github/copilot-sdk の issue をすべて見る
似ている issue
-
awaiting triage bug Causes friction Hop Gui P1 P2 Transforms
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
apache/flink-agents#1152 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
jenkinsci/blueocean-plugin#5417 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
objectionary/eo-graphs#75 ·