Add prerequisites field for workshop proposals (#342)

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Starefossen <968267+Starefossen@users.noreply.github.com>
Co-authored-by: Hans Kristian Flaatten <hans@flaatten.org>

Author: Copilot

__tests__/lib/proposal/schemas.test.ts

@@ -138,6 +138,60 @@ describe('ProposalInputSchema (strict)', () => {
     expect(result.success).toBe(true)
   })
 
+  it('accepts workshop format with prerequisites', () => {
+    const result = ProposalInputSchema.safeParse({
+      ...fullProposal,
+      format: Format.workshop_120,
+      capacity: 30,
+      prerequisites: 'Bring a computer with Docker installed',
+    })
+    expect(result.success).toBe(true)
+  })
+
+  it('accepts non-workshop format without prerequisites', () => {
+    const result = ProposalInputSchema.safeParse({
+      ...fullProposal,
+      format: Format.presentation_40,
+      prerequisites: undefined,
+    })
+    expect(result.success).toBe(true)
+  })
+
+  it('rejects prerequisites for non-workshop formats', () => {
+    const result = ProposalInputSchema.safeParse({
+      ...fullProposal,
+      format: Format.presentation_40,
+      prerequisites: 'Should not be allowed',
+    })
+    expect(result.success).toBe(false)
+  })
+
+  it('normalizes empty prerequisites to undefined', () => {
+    const result = ProposalInputSchema.safeParse({
+      ...fullProposal,
+      format: Format.workshop_120,
+      capacity: 30,
+      prerequisites: '   ',
+    })
+    expect(result.success).toBe(true)
+    if (result.success) {
+      expect(result.data.prerequisites).toBeUndefined()
+    }
+  })
+
+  it('trims whitespace from prerequisites', () => {
+    const result = ProposalInputSchema.safeParse({
+      ...fullProposal,
+      format: Format.workshop_120,
+      capacity: 30,
+      prerequisites: '  Docker required  ',
+    })
+    expect(result.success).toBe(true)
+    if (result.success) {
+      expect(result.data.prerequisites).toBe('Docker required')
+    }
+  })
+
   it('rejects missing title', () => {
     const { title: _, ...withoutTitle } = fullProposal
     const result = ProposalInputSchema.safeParse(withoutTitle)

sanity/schemaTypes/talk.ts

@@ -212,6 +212,23 @@ export default defineType({
             return true
           }),
     }),
+    defineField({
+      name: 'prerequisites',
+      title: 'Prerequisites',
+      type: 'text',
+      description:
+        'Prerequisites for workshop participants (e.g., "Bring a computer with Docker installed")',
+      hidden: ({ document }) => {
+        const format = document?.format as string | undefined
+        const prerequisites = document?.prerequisites as string | undefined
+        const isWorkshop =
+          format === 'workshop_120' || format === 'workshop_240'
+        return !(
+          isWorkshop ||
+          (prerequisites !== undefined && prerequisites !== null)
+        )
+      },
+    }),
     defineField({
       name: 'audienceFeedback',
       title: 'Audience Feedback',

src/components/cfp/ProposalForm.tsx

@@ -169,6 +169,7 @@ export function ProposalForm({
       audiences: proposal.audiences,
       tos: proposal.tos,
       capacity: proposal.capacity,
+      prerequisites: proposal.prerequisites,
       topics: topicRefs,
       speakers: allSpeakers.map((id) => ({
         _type: 'reference' as const,

src/components/program/TalkCard.tsx

@@ -10,7 +10,7 @@ import {
   XMarkIcon,
 } from '@heroicons/react/24/outline'
 import { TrackTalk } from '@/lib/conference/types'
-import { Status } from '@/lib/proposal/types'
+import { isWorkshopFormat, Status } from '@/lib/proposal/types'
 import { SpeakerAvatars } from '@/components/SpeakerAvatars'
 import { ClickableSpeakerNames } from '@/components/ClickableSpeakerNames'
 import { BookmarkButton } from '@/components/BookmarkButton'
@@ -436,6 +436,21 @@ export function TalkCard({
             </div>
           )}
 
+        {!compact &&
+          isConfirmed &&
+          talkData.format &&
+          isWorkshopFormat(talkData.format) &&
+          talkData.prerequisites && (
+            <div className="rounded-md border border-blue-200 bg-blue-50 p-4 dark:border-blue-800/50 dark:bg-blue-900/20">
+              <h4 className="text-sm font-semibold text-blue-900 dark:text-blue-200">
+                Prerequisites
+              </h4>
+              <p className="mt-1 text-sm whitespace-pre-wrap text-blue-800 dark:text-blue-300">
+                {talkData.prerequisites}
+              </p>
+            </div>
+          )}
+
         {!compact && !isConfirmed && !isWithdrawnOrRejected && (
           <div className="flex-1">
             <div className="text-sm text-gray-400 italic dark:text-gray-500">

src/components/proposal/ProposalDetailsForm.tsx

@@ -6,6 +6,7 @@ import {
   audiences as audiencesMap,
   Format,
   formats,
+  isWorkshopFormat,
   Language,
   languages,
   Level,
@@ -57,6 +58,17 @@ export function ProposalDetailsForm({
   )
   const [outline, setOutline] = useState(proposal?.outline ?? '')
   const [tos, setTos] = useState(proposal?.tos ?? false)
+  const [prerequisites, setPrerequisites] = useState(
+    proposal?.prerequisites ?? '',
+  )
+
+  // Clear prerequisites when format changes to non-workshop
+  useEffect(() => {
+    if (!isWorkshopFormat(format)) {
+      // eslint-disable-next-line react-hooks/set-state-in-effect -- Clear stale prerequisites when switching to non-workshop format
+      setPrerequisites('')
+    }
+  }, [format])
 
   // Push local state changes to parent
   useEffect(() => {
@@ -70,6 +82,7 @@ export function ProposalDetailsForm({
       topics,
       outline,
       tos,
+      prerequisites: prerequisites.trim() || undefined,
     })
   }, [
     title,
@@ -81,6 +94,7 @@ export function ProposalDetailsForm({
     topics,
     outline,
     tos,
+    prerequisites,
     setProposal,
   ])
 
@@ -202,6 +216,17 @@ export function ProposalDetailsForm({
               {outline || 'No outline provided'}
             </p>
           </div>
+
+          {isWorkshopFormat(format) && (
+            <div>
+              <label className="block text-sm font-medium text-gray-900 dark:text-white">
+                Prerequisites
+              </label>
+              <p className="mt-2 whitespace-pre-wrap text-gray-900 dark:text-white">
+                {prerequisites || 'No prerequisites specified'}
+              </p>
+            </div>
+          )}
         </div>
       ) : (
         <div className="mt-10 grid grid-cols-1 gap-x-6 gap-y-8 sm:grid-cols-6">
@@ -315,6 +340,23 @@ export function ProposalDetailsForm({
             </HelpText>
           </div>
 
+          {isWorkshopFormat(format) && (
+            <div className="col-span-full">
+              <Textarea
+                name="prerequisites"
+                label="Prerequisites"
+                rows={3}
+                value={prerequisites}
+                setValue={setPrerequisites}
+              />
+              <HelpText>
+                List any prerequisites participants should meet before attending
+                your workshop (e.g., &quot;Bring a computer with Docker
+                installed&quot;, &quot;Basic knowledge of Kubernetes&quot;).
+              </HelpText>
+            </div>
+          )}
+
           <div className="col-span-full">
             <Checkbox
               name="tos"

src/lib/proposal/types.ts

@@ -84,6 +84,7 @@ interface Proposal {
   tos: boolean
   video?: string
   capacity?: number
+  prerequisites?: string
   audienceFeedback?: AudienceFeedback
   attachments?: Attachment[]
 }

src/server/schemas/proposal.ts

@@ -51,6 +51,15 @@ const ProposalInputBaseSchema = z.object({
   }),
   video: z.string().nullable().optional().transform(nullToUndefined),
   capacity: z.number().nullable().optional().transform(nullToUndefined),
+  prerequisites: z
+    .string()
+    .nullable()
+    .optional()
+    .transform((val) => {
+      if (val === null || val === undefined) return undefined
+      const trimmed = val.trim()
+      return trimmed.length > 0 ? trimmed : undefined
+    }),
   speakers: z
     .array(ReferenceSchema)
     .nullable()
@@ -71,6 +80,18 @@ export const ProposalInputSchema = ProposalInputBaseSchema.refine(
     message: 'Workshop capacity is required for workshop formats',
     path: ['capacity'],
   },
+).refine(
+  (data) => {
+    // Prerequisites should only be provided for workshop formats
+    if (data.prerequisites && !isWorkshopFormat(data.format)) {
+      return false
+    }
+    return true
+  },
+  {
+    message: 'Prerequisites are only allowed for workshop formats',
+    path: ['prerequisites'],
+  },
 )
 
 // Admin-specific proposal creation (includes speaker IDs)