[TCSACR-655] Expose Session API to public by mbloch1 · Pull Request #7705 · Samsung/TizenFX

mbloch1 · 2026-06-16T16:06:43Z

Expose the Session API (currently platform-only) to public. See TCSACR-655, and ACR-1952 for the native side.

gemini-code-assist

Code Review

This pull request exposes the Session class and its events as public APIs by removing the [EditorBrowsable(EditorBrowsableState.Never)] attribute and adding <since_tizen>10.1</since_tizen> tags to several members. The feedback recommends adding the missing <since_tizen>10.1</since_tizen> XML documentation tags to the Session class and its events (AddUserWait, RemoveUserWait, SwitchUserWait, and SwitchUserCompleted). Additionally, it suggests removing the [EditorBrowsable(EditorBrowsableState.Never)] attribute from the SubsessionEventArgs classes to ensure they are visible in IDE IntelliSense now that they are publicly exposed.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

TizenAPI-Bot · 2026-06-16T16:18:00Z

Public API Changed

Please follow the ACR process for the changed API below.

Added: 15, Removed: 0, Changed: 0

Added

+ /// <since_tizen>none</since_tizen
+ Tizen.System.Session

+ /// <since_tizen>10.1</since_tizen
+ static System.Int32 Tizen.System.Session::MaxUserLength

+ /// <since_tizen>10.1</since_tizen
+ static System.String Tizen.System.Session::EmptyUser

+ /// <since_tizen>10.1</since_tizen
+ System.Int32 Tizen.System.Session::SessionUID()

+ /// <since_tizen>10.1</since_tizen
+ System.Collections.Generic.IReadOnlyList`1<System.String> Tizen.System.Session::GetUsers()

+ /// <since_tizen>10.1</since_tizen
+ System.String Tizen.System.Session::GetCurrentUser()

+ /// <since_tizen>10.1</since_tizen
+ System.Threading.Tasks.Task Tizen.System.Session::SubsessionAddUserAsync(System.String)

+ /// <since_tizen>10.1</since_tizen
+ System.Threading.Tasks.Task Tizen.System.Session::SubsessionRemoveUserAsync(System.String)

+ /// <since_tizen>10.1</since_tizen
+ System.Threading.Tasks.Task Tizen.System.Session::SubsessionSwitchUserAsync(System.String)

+ /// <since_tizen>10.1</since_tizen
+ System.Void Tizen.System.Session::SubsessionEventMarkAsDone(Tizen.System.SubsessionEventArgs)

+ /// <since_tizen>10.1</since_tizen
+ static Tizen.System.Session Tizen.System.Session::GetInstance(System.Int32)

+ /// <since_tizen>none</since_tizen
+ System.EventHandler`1<Tizen.System.AddUserEventArgs> Tizen.System.Session::AddUserWait

+ /// <since_tizen>none</since_tizen
+ System.EventHandler`1<Tizen.System.RemoveUserEventArgs> Tizen.System.Session::RemoveUserWait

+ /// <since_tizen>none</since_tizen
+ System.EventHandler`1<Tizen.System.SwitchUserCompletionEventArgs> Tizen.System.Session::SwitchUserCompleted

+ /// <since_tizen>none</since_tizen
+ System.EventHandler`1<Tizen.System.SwitchUserWaitEventArgs> Tizen.System.Session::SwitchUserWait

Internal API Changed

Added: 0, Removed: 15, Changed: 6

Removed

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- Tizen.System.Session

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- static System.Int32 Tizen.System.Session::MaxUserLength

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- static System.String Tizen.System.Session::EmptyUser

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Int32 Tizen.System.Session::SessionUID()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Collections.Generic.IReadOnlyList`1<System.String> Tizen.System.Session::GetUsers()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.String Tizen.System.Session::GetCurrentUser()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Threading.Tasks.Task Tizen.System.Session::SubsessionAddUserAsync(System.String)

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Threading.Tasks.Task Tizen.System.Session::SubsessionRemoveUserAsync(System.String)

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Threading.Tasks.Task Tizen.System.Session::SubsessionSwitchUserAsync(System.String)

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Void Tizen.System.Session::SubsessionEventMarkAsDone(Tizen.System.SubsessionEventArgs)

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- static Tizen.System.Session Tizen.System.Session::GetInstance(System.Int32)

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.EventHandler`1<Tizen.System.AddUserEventArgs> Tizen.System.Session::AddUserWait

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.EventHandler`1<Tizen.System.RemoveUserEventArgs> Tizen.System.Session::RemoveUserWait

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.EventHandler`1<Tizen.System.SwitchUserCompletionEventArgs> Tizen.System.Session::SwitchUserCompleted

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.EventHandler`1<Tizen.System.SwitchUserWaitEventArgs> Tizen.System.Session::SwitchUserWait

Changed

- /// <since_tizen>none</since_tizen
+ /// <since_tizen>10.1</since_tizen
[EditorBrowsable(EditorBrowsableState.Never)]
Tizen.System.AddUserEventArgs

- /// <since_tizen>none</since_tizen
+ /// <since_tizen>10.1</since_tizen
[EditorBrowsable(EditorBrowsableState.Never)]
Tizen.System.RemoveUserEventArgs

- /// <since_tizen>none</since_tizen
+ /// <since_tizen>10.1</since_tizen
[EditorBrowsable(EditorBrowsableState.Never)]
Tizen.System.SubsessionEventArgs

- /// <since_tizen>none</since_tizen
+ /// <since_tizen>10.1</since_tizen
[EditorBrowsable(EditorBrowsableState.Never)]
Tizen.System.SwitchUserCompletionEventArgs

- /// <since_tizen>none</since_tizen
+ /// <since_tizen>10.1</since_tizen
[EditorBrowsable(EditorBrowsableState.Never)]
Tizen.System.SwitchUserEventArgs

- /// <since_tizen>none</since_tizen
+ /// <since_tizen>10.1</since_tizen
[EditorBrowsable(EditorBrowsableState.Never)]
Tizen.System.SwitchUserWaitEventArgs

JoonghyunCho · 2026-06-17T23:06:09Z

🤖 [AI Review]
Left 2 inline comments (🔴 2, 🟡 0). Both flag newly-public members exposed by removing [EditorBrowsable(Never)] that are missing the required <since_tizen> tag: the Session class and the four events (AddUserWait, RemoveUserWait, SwitchUserWait, SwitchUserCompleted). All other members in this diff correctly received <since_tizen>10.1</since_tizen>.

Automated review by AI assistant

See ACR-1952.

mbloch1 · 2026-06-18T17:30:56Z

Addressed comments, <since_tizen> is now added consistently wherever [EditorBrowsable(Never)] was removed.

TizenAPI-Bot · 2026-06-18T17:34:46Z

Public API Changed

Please follow the ACR process for the changed API below.

Added: 24, Removed: 0, Changed: 0

Added

+ /// <since_tizen>10.1</since_tizen
+ Tizen.System.AddUserEventArgs

+ /// <since_tizen>10.1</since_tizen
+ System.String Tizen.System.AddUserEventArgs::UserName()

+ /// <since_tizen>10.1</since_tizen
+ Tizen.System.RemoveUserEventArgs

+ /// <since_tizen>10.1</since_tizen
+ System.String Tizen.System.RemoveUserEventArgs::UserName()

+ /// <since_tizen>10.1</since_tizen
+ Tizen.System.Session

+ /// <since_tizen>10.1</since_tizen
+ static System.Int32 Tizen.System.Session::MaxUserLength

+ /// <since_tizen>10.1</since_tizen
+ static System.String Tizen.System.Session::EmptyUser

+ /// <since_tizen>10.1</since_tizen
+ System.Int32 Tizen.System.Session::SessionUID()

+ /// <since_tizen>10.1</since_tizen
+ System.Collections.Generic.IReadOnlyList`1<System.String> Tizen.System.Session::GetUsers()

+ /// <since_tizen>10.1</since_tizen
+ System.String Tizen.System.Session::GetCurrentUser()

+ /// <since_tizen>10.1</since_tizen
+ System.Void Tizen.System.Session::SubsessionEventMarkAsDone(Tizen.System.SubsessionEventArgs)

+ /// <since_tizen>10.1</since_tizen
+ static Tizen.System.Session Tizen.System.Session::GetInstance(System.Int32)

+ /// <since_tizen>10.1</since_tizen
+ System.EventHandler`1<Tizen.System.AddUserEventArgs> Tizen.System.Session::AddUserWait

+ /// <since_tizen>10.1</since_tizen
+ System.EventHandler`1<Tizen.System.RemoveUserEventArgs> Tizen.System.Session::RemoveUserWait

+ /// <since_tizen>10.1</since_tizen
+ System.EventHandler`1<Tizen.System.SwitchUserCompletionEventArgs> Tizen.System.Session::SwitchUserCompleted

+ /// <since_tizen>10.1</since_tizen
+ System.EventHandler`1<Tizen.System.SwitchUserWaitEventArgs> Tizen.System.Session::SwitchUserWait

+ /// <since_tizen>10.1</since_tizen
+ Tizen.System.SubsessionEventArgs

+ /// <since_tizen>10.1</since_tizen
+ System.Int32 Tizen.System.SubsessionEventArgs::SessionUID()

+ /// <since_tizen>10.1</since_tizen
+ Tizen.System.SwitchUserCompletionEventArgs

+ /// <since_tizen>10.1</since_tizen
+ Tizen.System.SwitchUserEventArgs

+ /// <since_tizen>10.1</since_tizen
+ System.Int64 Tizen.System.SwitchUserEventArgs::SwitchID()

+ /// <since_tizen>10.1</since_tizen
+ System.String Tizen.System.SwitchUserEventArgs::UserNameNext()

+ /// <since_tizen>10.1</since_tizen
+ System.String Tizen.System.SwitchUserEventArgs::UserNamePrev()

+ /// <since_tizen>10.1</since_tizen
+ Tizen.System.SwitchUserWaitEventArgs

Internal API Changed

Added: 0, Removed: 24, Changed: 0

Removed

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- Tizen.System.AddUserEventArgs

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.String Tizen.System.AddUserEventArgs::UserName()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- Tizen.System.RemoveUserEventArgs

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.String Tizen.System.RemoveUserEventArgs::UserName()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- Tizen.System.Session

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- static System.Int32 Tizen.System.Session::MaxUserLength

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- static System.String Tizen.System.Session::EmptyUser

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Int32 Tizen.System.Session::SessionUID()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Collections.Generic.IReadOnlyList`1<System.String> Tizen.System.Session::GetUsers()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.String Tizen.System.Session::GetCurrentUser()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Void Tizen.System.Session::SubsessionEventMarkAsDone(Tizen.System.SubsessionEventArgs)

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- static Tizen.System.Session Tizen.System.Session::GetInstance(System.Int32)

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.EventHandler`1<Tizen.System.AddUserEventArgs> Tizen.System.Session::AddUserWait

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.EventHandler`1<Tizen.System.RemoveUserEventArgs> Tizen.System.Session::RemoveUserWait

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.EventHandler`1<Tizen.System.SwitchUserCompletionEventArgs> Tizen.System.Session::SwitchUserCompleted

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.EventHandler`1<Tizen.System.SwitchUserWaitEventArgs> Tizen.System.Session::SwitchUserWait

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- Tizen.System.SubsessionEventArgs

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Int32 Tizen.System.SubsessionEventArgs::SessionUID()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- Tizen.System.SwitchUserCompletionEventArgs

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- Tizen.System.SwitchUserEventArgs

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Int64 Tizen.System.SwitchUserEventArgs::SwitchID()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.String Tizen.System.SwitchUserEventArgs::UserNameNext()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.String Tizen.System.SwitchUserEventArgs::UserNamePrev()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- Tizen.System.SwitchUserWaitEventArgs

JoonghyunCho · 2026-06-18T23:06:42Z

        /// <exception cref="UnauthorizedAccessException">Not permitted</exception>
        /// <exception cref="NotSupportedException">Not supported</exception>
-        [EditorBrowsable(EditorBrowsableState.Never)]
+        /// <since_tizen>10.1</since_tizen>


🤖 [AI Review]
🔴 Critical: GetUsers() is now part of the public API surface (the [EditorBrowsable(...Never)] attribute was removed), but its XML doc is missing a <returns> tag for its non-void return value.

Suggested change

/// <since_tizen>10.1</since_tizen>

/// <returns>A read-only list of all subsession user IDs for this session.</returns>

/// <since_tizen>10.1</since_tizen>

JoonghyunCho · 2026-06-18T23:06:48Z

        /// <exception cref="UnauthorizedAccessException">Not permitted</exception>
        /// <exception cref="NotSupportedException">Not supported</exception>
-        [EditorBrowsable(EditorBrowsableState.Never)]
+        /// <since_tizen>10.1</since_tizen>


🤖 [AI Review]
🔴 Critical: GetCurrentUser() is now part of the public API surface (the [EditorBrowsable(...Never)] attribute was removed), but its XML doc is missing a <returns> tag for its non-void return value.

Suggested change

/// <since_tizen>10.1</since_tizen>

/// <returns>The currently active subsession user ID, or "" (EmptyUser) when no subsession is enabled.</returns>

/// <since_tizen>10.1</since_tizen>

JoonghyunCho · 2026-06-18T23:07:00Z

🤖 [AI Review]
Reviewed the new commits exposing the Session API to public. Left 2 inline comments (🔴 2 critical, 🟡 0 suggestion).

GetUsers() and GetCurrentUser() are now public (the [EditorBrowsable(...Never)] attribute was removed) but lack a <returns> tag.
The <since_tizen>/<summary> additions on the class, fields, events, and GetInstance/SubsessionEventMarkAsDone are otherwise complete; members still marked [EditorBrowsable(...Never)] (e.g. Subsession*Async, CurrentUID) remain out of the documented surface, consistent with the "partially public" intent.

See inline for details.

Automated review by AI assistant — final merge decision rests with human reviewers.

mbloch1 · 2026-06-19T13:53:25Z

Addressed comments.
Made a ticket https://jira.sec.samsung.net/projects/TCSACR/issues/TCSACR-655

TizenAPI-Bot · 2026-06-19T13:58:48Z

Public API Changed

Please follow the ACR process for the changed API below.

Added: 24, Removed: 0, Changed: 0

Added

+ /// <since_tizen>10.1</since_tizen
+ Tizen.System.AddUserEventArgs

+ /// <since_tizen>10.1</since_tizen
+ System.String Tizen.System.AddUserEventArgs::UserName()

+ /// <since_tizen>10.1</since_tizen
+ Tizen.System.RemoveUserEventArgs

+ /// <since_tizen>10.1</since_tizen
+ System.String Tizen.System.RemoveUserEventArgs::UserName()

+ /// <since_tizen>10.1</since_tizen
+ Tizen.System.Session

+ /// <since_tizen>10.1</since_tizen
+ static System.Int32 Tizen.System.Session::MaxUserLength

+ /// <since_tizen>10.1</since_tizen
+ static System.String Tizen.System.Session::EmptyUser

+ /// <since_tizen>10.1</since_tizen
+ System.Int32 Tizen.System.Session::SessionUID()

+ /// <since_tizen>10.1</since_tizen
+ System.Collections.Generic.IReadOnlyList`1<System.String> Tizen.System.Session::GetUsers()

+ /// <since_tizen>10.1</since_tizen
+ System.String Tizen.System.Session::GetCurrentUser()

+ /// <since_tizen>10.1</since_tizen
+ System.Void Tizen.System.Session::SubsessionEventMarkAsDone(Tizen.System.SubsessionEventArgs)

+ /// <since_tizen>10.1</since_tizen
+ static Tizen.System.Session Tizen.System.Session::GetInstance(System.Int32)

+ /// <since_tizen>10.1</since_tizen
+ System.EventHandler`1<Tizen.System.AddUserEventArgs> Tizen.System.Session::AddUserWait

+ /// <since_tizen>10.1</since_tizen
+ System.EventHandler`1<Tizen.System.RemoveUserEventArgs> Tizen.System.Session::RemoveUserWait

+ /// <since_tizen>10.1</since_tizen
+ System.EventHandler`1<Tizen.System.SwitchUserCompletionEventArgs> Tizen.System.Session::SwitchUserCompleted

+ /// <since_tizen>10.1</since_tizen
+ System.EventHandler`1<Tizen.System.SwitchUserWaitEventArgs> Tizen.System.Session::SwitchUserWait

+ /// <since_tizen>10.1</since_tizen
+ Tizen.System.SubsessionEventArgs

+ /// <since_tizen>10.1</since_tizen
+ System.Int32 Tizen.System.SubsessionEventArgs::SessionUID()

+ /// <since_tizen>10.1</since_tizen
+ Tizen.System.SwitchUserCompletionEventArgs

+ /// <since_tizen>10.1</since_tizen
+ Tizen.System.SwitchUserEventArgs

+ /// <since_tizen>10.1</since_tizen
+ System.Int64 Tizen.System.SwitchUserEventArgs::SwitchID()

+ /// <since_tizen>10.1</since_tizen
+ System.String Tizen.System.SwitchUserEventArgs::UserNameNext()

+ /// <since_tizen>10.1</since_tizen
+ System.String Tizen.System.SwitchUserEventArgs::UserNamePrev()

+ /// <since_tizen>10.1</since_tizen
+ Tizen.System.SwitchUserWaitEventArgs

Internal API Changed

Added: 0, Removed: 24, Changed: 0

Removed

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- Tizen.System.AddUserEventArgs

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.String Tizen.System.AddUserEventArgs::UserName()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- Tizen.System.RemoveUserEventArgs

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.String Tizen.System.RemoveUserEventArgs::UserName()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- Tizen.System.Session

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- static System.Int32 Tizen.System.Session::MaxUserLength

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- static System.String Tizen.System.Session::EmptyUser

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Int32 Tizen.System.Session::SessionUID()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Collections.Generic.IReadOnlyList`1<System.String> Tizen.System.Session::GetUsers()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.String Tizen.System.Session::GetCurrentUser()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Void Tizen.System.Session::SubsessionEventMarkAsDone(Tizen.System.SubsessionEventArgs)

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- static Tizen.System.Session Tizen.System.Session::GetInstance(System.Int32)

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.EventHandler`1<Tizen.System.AddUserEventArgs> Tizen.System.Session::AddUserWait

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.EventHandler`1<Tizen.System.RemoveUserEventArgs> Tizen.System.Session::RemoveUserWait

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.EventHandler`1<Tizen.System.SwitchUserCompletionEventArgs> Tizen.System.Session::SwitchUserCompleted

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.EventHandler`1<Tizen.System.SwitchUserWaitEventArgs> Tizen.System.Session::SwitchUserWait

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- Tizen.System.SubsessionEventArgs

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Int32 Tizen.System.SubsessionEventArgs::SessionUID()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- Tizen.System.SwitchUserCompletionEventArgs

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- Tizen.System.SwitchUserEventArgs

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.Int64 Tizen.System.SwitchUserEventArgs::SwitchID()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.String Tizen.System.SwitchUserEventArgs::UserNameNext()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- System.String Tizen.System.SwitchUserEventArgs::UserNamePrev()

- /// <since_tizen>none</since_tizen
- [EditorBrowsable(EditorBrowsableState.Never)]
- Tizen.System.SwitchUserWaitEventArgs

JoonghyunCho · 2026-06-21T23:05:44Z

🤖 [AI Review]

Reviewed — no findings. (delta commit eea3df89)

Scope checked:

Verified the new commit adds <returns> tags to GetUsers() and GetCurrentUser(), resolving the previously flagged missing-documentation findings.
Confirmed both members now carry complete XML docs (<summary>, <returns>, <exception>, <since_tizen>).
Confirmed this delta is documentation-only; no method signature or runtime behavior changed.

No 🔴 critical issues, no 🟡 suggestions to flag.

Automated review — final merge decision rests with human reviewers.

mbloch1 requested a review from chanwoochoi as a code owner June 16, 2026 16:06

github-actions Bot added the API15 label Jun 16, 2026

mbloch1 changed the title ~~Public api 2026 06~~ [ACR-1952] Expose Session API to public Jun 16, 2026

gemini-code-assist Bot reviewed Jun 16, 2026

View reviewed changes

TizenAPI-Bot added ACR Required Internal API Changed labels Jun 16, 2026

JoonghyunCho reviewed Jun 17, 2026

View reviewed changes

Comment thread src/Tizen.System.Session/Session/Session.cs

JoonghyunCho reviewed Jun 17, 2026

View reviewed changes

Comment thread src/Tizen.System.Session/Session/Session.cs

mbloch1 added 2 commits June 18, 2026 19:28

Annotate Session functions with since_tizen

1b7db68

Make Session API partially public

0a5c2a5

See ACR-1952.

mbloch1 force-pushed the public-api-2026-06 branch from e5dc104 to 0a5c2a5 Compare June 18, 2026 17:29

JoonghyunCho reviewed Jun 18, 2026

View reviewed changes

Sessiond API: add <returns> tags.

eea3df8

mbloch1 changed the title ~~[ACR-1952] Expose Session API to public~~ [TCSACR-655] Expose Session API to public Jun 19, 2026

	/// <since_tizen>10.1</since_tizen>
	/// <returns>A read-only list of all subsession user IDs for this session.</returns>
	/// <since_tizen>10.1</since_tizen>

	/// <since_tizen>10.1</since_tizen>
	/// <returns>The currently active subsession user ID, or "" (EmptyUser) when no subsession is enabled.</returns>
	/// <since_tizen>10.1</since_tizen>

Uh oh!

Conversation

mbloch1 commented Jun 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

TizenAPI-Bot commented Jun 16, 2026

Public API Changed

Added

Internal API Changed

Removed

Changed

Uh oh!

Uh oh!

Uh oh!

JoonghyunCho commented Jun 17, 2026

Uh oh!

mbloch1 commented Jun 18, 2026

Uh oh!

TizenAPI-Bot commented Jun 18, 2026

Public API Changed

Added

Internal API Changed

Removed

Uh oh!

JoonghyunCho Jun 18, 2026

Choose a reason for hiding this comment

Uh oh!

JoonghyunCho Jun 18, 2026

Choose a reason for hiding this comment

Uh oh!

JoonghyunCho commented Jun 18, 2026

Uh oh!

mbloch1 commented Jun 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

TizenAPI-Bot commented Jun 19, 2026

Public API Changed

Added

Internal API Changed

Removed

Uh oh!

JoonghyunCho commented Jun 21, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

mbloch1 commented Jun 16, 2026 •

edited

Loading

mbloch1 commented Jun 19, 2026 •

edited

Loading